diff options
-rw-r--r-- | src/com/android/calendarcommon/RecurrenceProcessor.java | 5 | ||||
-rw-r--r-- | tests/src/com/android/calendarcommon/RecurrenceProcessorTest.java | 2 |
2 files changed, 3 insertions, 4 deletions
diff --git a/src/com/android/calendarcommon/RecurrenceProcessor.java b/src/com/android/calendarcommon/RecurrenceProcessor.java index c3ebc29..577ac49 100644 --- a/src/com/android/calendarcommon/RecurrenceProcessor.java +++ b/src/com/android/calendarcommon/RecurrenceProcessor.java @@ -747,9 +747,8 @@ bysetpos: // inside the loop. It is important that count gets incremented exactly // once here or in the loop for dtstart. // - // TODO: this is only correct if DTSTART is synchronized with the recurrence rule. - // If e.g. DSTART is 2011/07/01 (a Friday), and the rule includes only "BYDAY=SA,SU", - // this will insert an incorrect entry. (Why is this being done?) + // NOTE: if DTSTART is not synchronized with the recurrence rule, the first instance + // we return will not fit the RRULE pattern. if (add && dtstartDateValue >= rangeStartDateValue && dtstartDateValue < rangeEndDateValue) { out.add(dtstartDateValue); diff --git a/tests/src/com/android/calendarcommon/RecurrenceProcessorTest.java b/tests/src/com/android/calendarcommon/RecurrenceProcessorTest.java index 2ca2d35..21d2d52 100644 --- a/tests/src/com/android/calendarcommon/RecurrenceProcessorTest.java +++ b/tests/src/com/android/calendarcommon/RecurrenceProcessorTest.java @@ -378,7 +378,7 @@ public class RecurrenceProcessorTest extends TestCase { null /* rdate */, null /* exrule */, null /* exdate */, "20060101T000000", "20101231T000000", new String[]{ - "20060101T100000", // TODO: this is wrong (bug in recurrence gen) + "20060101T100000", "20060113T100000", "20061013T100000", "20070413T100000", |