Mikhail Gusarov wrote:
> Oops, I forgot to mail updated patch, last one had two bugs:
> - 'fire_event_deleted' instead of 'fire_event_removed' once.
> - 'fire_event_removed' may be called both with 'event' argument and
>   without one, depending on the context, so let it accept and pass
>   further optional 'event' argument.
> See the fix in attach.
> ps: I'm subscribed to gajim-devel, no need to copy answer to my address.
> pps: Probably it worth to factor out publisher/subsriber snippet to
>      the separate class and use it in everywhere. I'd do it if you
>      agree that's a good idea.

to be more general, fire_event_removed get and transmit a list of
removed events.

about your pps: you want systray (and other classes that would use this
event system) to be a child of a new class that will contain some
general code to register / unregister listener ? that sounds a good idea

Gajim-devel mailing list

Reply via email to