Hi Lalo, Could you submit this as a trac ticket with the patch?
Thanks, Beau On 20-May-07, at 1:11 PM, Lalo Martins wrote: > > Hi. > > Straight to the point: I'd like to make a case for a __disconnect__ > interface, like __connect__. > > My use case is: I have a small library that wraps Google, Yahoo, etc > maps. I'm very happy that I don't need to listen for events I don't > need; I can call GEvent.bind() or its analogue on the __connect__ > method. But I never "unlisten", since there is no __disconnect__. > (Actually, I did an ugly hack on the event handler to check if the > event > is disconnected, but that's not quite ideal.) > > Patch follows: > =================================================================== > --- MochiKit/Signal.js (revision 1292) > +++ MochiKit/Signal.js (working copy) > @@ -695,11 +695,14 @@ > return; > } > ident.connected = false; > + var src = ident.source; > // check isDOM > if (!ident.isDOM) { > + if(typeof(src.__disconnect__) == 'function') { > + src.__disconnect__(ident); > + } > return; > } > - var src = ident.source; > var sig = ident.signal; > var listener = ident.listener; > =================================================================== > > > best, > Lalo Martins > -- > So many of our dreams at first seem impossible, > then they seem improbable, and then, when we > summon the will, they soon become inevitable. > ----- > personal: http://lalo.hystericalraisins.net/ > technical: http://www.hystericalraisins.net/ > GNU: never give up freedom http://www.gnu.org/ > > > > --~--~---------~--~----~------------~-------~--~----~ You received this message because you are subscribed to the Google Groups "MochiKit" group. To post to this group, send email to mochikit@googlegroups.com To unsubscribe from this group, send email to [EMAIL PROTECTED] For more options, visit this group at http://groups.google.com/group/mochikit?hl=en -~----------~----~----~----~------~----~------~--~---