Code review comment for lp:~pkunal-parmar/ubuntu-calendar-app/today

Revision history for this message
Olivier Tilloy (osomon) wrote :

You could simplify the code further by completely getting rid of the 'goToNextMonth' parameter, using the sign of 'count' as a multiplier (to get the sign of count, just use cmp(count, 0)).

That’s only a suggestion though, feel free to ignore.

« Back to merge proposal