c508f9ac

By: Tom Sydney Kerckhove <syd@cs-syd.eu>

ical-recurrence: keep the week day of a numeric ByDay

@
The BYDAY rule part
MUST NOT be specified with a numeric value with the FREQ rule part
set to YEARLY when the BYWEEKNO rule part is specified.
@

The number is meaningless once BYWEEKNO has fixed the week, so discard
the number and keep the week day rather than discarding the rule part.

Dropping it entirely was worse than ignoring the rule part: with no week
days left, the BYWEEKNO branch of yearlyDayCandidate falls back to all of
Monday to Sunday, which widens the recurrence set to every day of the
week instead of narrowing it.

Suite timing

Time to Start Worker time Duration Time to finish Idle
Config 4m56s 1s 1s 4m57s 4m56s
Eval 5m00s 20s 20s 5m21s 3s
Build 5m20s 9m13s 7m09s 12m30s 41s
Suite 4m56s 9m35s 7m34s 12m30s 5m41s

Timeline

0s7m8m9m10m11m12m