Code review comment for lp:~jjardon/indicator-datetime/fix-740977

Revision history for this message
Antti Kaijanmäki (kaijanmaki) wrote :

Thank you for your time and energy Javier to make Ubuntu better!

You are correct that the timer is not needed anymore, but I suggest we don't remove the timer from Oneiric code at this late of the cycle as the timer does not introduce any regression or unwanted behavior.

I'm now working on upgrading the code of indicator-datetime for Precise to use the newer EDS API and will in the process also clean up all the unnecessary old code, including the timer.

« Back to merge proposal