[sugar] [PATCH] Sort activities in the list and ring by installation date

Tomeu Vizoso tomeu
Wed May 28 06:38:02 EDT 2008


On Wed, May 28, 2008 at 12:30 PM, Simon Schampijer <simon at schampijer.de> wrote:
>
> I would prefer to have chars to differentiate between two objects:
> def _compare_activities(self, icon_1, icon_2)
> def _compare_activities(self, icon_a, icon_b)

Ok ;)

> Would love to see tomeu to run pylint before submitting patches!
> E0602: 92:ActivitiesRing.__activity_added_cb: Undefined variable 'info'

Sorry, will give one more look into pylint.

> General question:
> We have getters in ActivityEntry for installation_time and some other
> variables. We don't make them public variables because we want read access
> only right?

Yes, that's the idea.

Thanks,

Tomeu



More information about the Sugar-devel mailing list