LGTM, should work fine.
Only minor suggestion is we pull limitDate() 4x in a small paragraph of code.
If QOrganizerRecurrenceRule does something dumb like recalculating it each time limitDate() is called, be better call it just once and cache it in a const temporary.
« Back to merge proposal
LGTM, should work fine.
Only minor suggestion is we pull limitDate() 4x in a small paragraph of code.
If QOrganizerRecur renceRule does something dumb like recalculating it each time limitDate() is called, be better call it just once and cache it in a const temporary.