Code review comment for lp:~ertainl/alarm-clock/bug1122500

Revision history for this message
Johannes H. Jensen (joh) wrote :

Looks good! The include_today flag was added to guard against a corner case, but the call to alarm_time_is_future() seems to make it obsolete.

review: Approve

« Back to merge proposal