Re: [Gajim-devel] [PATCH] publisher/subscriber in common/events.py (+ decoupling from systray.py)

2007-03-20 Thread Yann Le Boulanger
Mikhail Gusarov wrote: > Twas brillig at 10:05:06 11.03.2007 UTC+01 when Yann Le Boulanger did gyre > and gimble: > > YLB> about your pps: you want systray (and other classes that would use this > YLB> event system) to be a child of a new class that will contain some > general > YLB> code to

Re: [Gajim-devel] [PATCH] publisher/subscriber in common/events.py (+ decoupling from systray.py)

2007-03-19 Thread Mikhail Gusarov
Twas brillig at 10:05:06 11.03.2007 UTC+01 when Yann Le Boulanger did gyre and gimble: YLB> about your pps: you want systray (and other classes that would use this YLB> event system) to be a child of a new class that will contain some general YLB> code to register / unregister listener ? tha

Re: [Gajim-devel] [PATCH] publisher/subscriber in common/events.py (+ decoupling from systray.py)

2007-03-11 Thread Yann Le Boulanger
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 >

Re: [Gajim-devel] [PATCH] publisher/subscriber in common/events.py (+ decoupling from systray.py)

2007-03-10 Thread Mikhail Gusarov
Twas brillig at 12:29:16 10.03.2007 UTC+01 when yann did gyre and gimble: y> Thanks for the patch. It's a good solution to handle events better y> in our code. I've modified your code a bit (integrate y> SystrayEventsListener class in Systray one) and commited it in rev y> 8008 Oops, I forgo

Re: [Gajim-devel] [PATCH] publisher/subscriber in common/events.py (+ decoupling from systray.py)

2007-03-10 Thread yann
Mikhail Gusarov a écrit : > Hi, > > Attached is the patch which allows subscribing to events in > common/events.py (using publisher/subscriber pattern). As a > side-effect, this patch decouples systray.py and common/events.py > (common/events.py no longer even imports common.gajim). > > The patch

[Gajim-devel] [PATCH] publisher/subscriber in common/events.py (+ decoupling from systray.py)

2007-03-08 Thread Mikhail Gusarov
Hi, Attached is the patch which allows subscribing to events in common/events.py (using publisher/subscriber pattern). As a side-effect, this patch decouples systray.py and common/events.py (common/events.py no longer even imports common.gajim). The patch allows additional modules to be written,