Code review comment for lp:~charlesk/indicator-datetime/lp-1204532

Revision history for this message
Charles Kerr (charlesk) wrote :

> the menu refreshes, but the 5 appointments stay the same.

Fixed r256

> * In on_appointments_change() you should probably handle the case of a GDateTime being NULL. I think that the indicator functions are fine, but the g_datetime_unref() is not.

Fixed r257

« Back to merge proposal