On Wed, May 28, 2008 at 12:30 PM, Simon Schampijer <[EMAIL PROTECTED]> 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
_______________________________________________
Sugar mailing list
[email protected]
http://lists.laptop.org/listinfo/sugar

Reply via email to