[sugar] [PATCH] Update only when visible (Journal)

Tomeu Vizoso tomeu at tomeuvizoso.net
Wed May 21 07:42:06 EDT 2008


On Wed, May 21, 2008 at 12:51 PM, Simon Schampijer <simon at schampijer.de> wrote:
> Hi Tomeu,
>
> Patch looks great and should really be an improvement. Not sure we need the
> debug messages for the timers logging.debug('Remove date updating timer').
>
> As a general note running a pylint for the journal would be nice as well
> (should do that for the browse now as well :). Though pylint output is not
> perfect yet it improves quality a bit.
>
> r+

Agreed, will do a pylint check after this commit.

Thanks!

Tomeu


More information about the Sugar mailing list