[Dextrose] [PATCH] Prevent multiple XS icon when re register
David Farning
dfarning at gmail.com
Mon Aug 9 12:57:14 EDT 2010
On Sat, Aug 7, 2010 at 8:16 PM, Martin Abente <mabente at paraguayeduca.org> wrote:
> On Fri, 06 Aug 2010 21:18:37 -0400, Bernie Innocenti <bernie at codewiz.org>
> wrote:
>> El Fri, 06-08-2010 a las 14:59 -0400, Martin Abente escribió:
>>> @@ -122,7 +127,7 @@ class VolumesToolbar(gtk.Toolbar):
>>> def _get_button_for_mount(self, mount):
>>> mount_point = mount.get_root().get_path()
>>> for button in self.get_children():
>>> - if type(button) == VolumeButton and \
>>> + if button.__class__ == VolumeButton and \
>>
>> I do not understand why this change was necessary: doesn't type(obj)
>> always return obj.__class__?
>>
>
> type does not work for custom classes.
>
>> Maybe isinstance() would be more appropriate here.
>
> Ops, forgot about that, we could change it directly into the patch maybe?
Martin,
Can you submit a new corrected patch? I am still learning the workflow.,
david
More information about the Dextrose
mailing list