Jason Tackaberry wrote:
>              except (KeyboardInterrupt, SystemExit):
> +                # FIXME: kaa.metadata is a library, and it is not its place
> +                # to institute behaviour on these exceptions.  Rather it
> +                # should bubble up these exceptions and let the caller
> +                # deal with them.
>                  sys.exit(0)

The only reason for SystemExit is sys.exit but you are right, it is
ugly to send it this way. But KeyboardInterrupt should be the only
thing that could happen inside this block. What about:

| except (KeyboardInterrupt, SystemExit), e:
|     raise e

That should work as expected.


Dischi

-- 
Beat me, whip me, make me use Windows!

Attachment: pgpDsjDpKGNBd.pgp
Description: PGP signature

-------------------------------------------------------------------------
Take Surveys. Earn Cash. Influence the Future of IT
Join SourceForge.net's Techsay panel and you'll get the chance to share your
opinions on IT & business topics through brief surveys-and earn cash
http://www.techsay.com/default.php?page=join.php&p=sourceforge&CID=DEVDEV
_______________________________________________
Freevo-cvslog mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/freevo-cvslog

Reply via email to