Jason Tackaberry wrote:
>     kaa.signals['exception'].connect(exception_handler)

I'm not sure the exception handler is a signal. It would be very
confusing if there would be more than one handler. IMHO it should be a
simple callback provided to main.run

> I also think that everywhere else in kaa that we're emitting on an
> exception signal (for example Thread exception signal, InProgress
> exception, etc), we should emit type, value, and traceback args for the
> current exception.  Like this:
>
>     try:
>        do_callback()
>     except Exception, exc:
>       self.signals['exception'].emit(exc.__class__, exc, sys.exc_traceback)

We should make sure not tp break too much by adding this.


Dischi

-- 
Anything labeled "NEW" and/or "IMPROVED" isn't.  The label means the
price went up.  The label "ALL NEW", "COMPLETELY NEW", or "GREAT NEW"
means the price went way up.

Attachment: pgpYWS0mK0wYh.pgp
Description: PGP signature

-------------------------------------------------------------------------
Check out the new SourceForge.net Marketplace.
It's the best place to buy or sell services for
just about anything Open Source.
http://ad.doubleclick.net/clk;164216239;13503038;w?http://sf.net/marketplace
_______________________________________________
Freevo-devel mailing list
Freevo-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/freevo-devel

Reply via email to