This issue tracker has been migrated to GitHub, and is currently read-only.
For more information, see the GitHub FAQs in the Python's Developer Guide.

Author vstinner
Recipients serhiy.storchaka, vstinner
Date 2015-03-06.22:22:22
SpamBayes Score -1.0
Marked as misclassified Yes
Message-id <CAMpsgwZ+QV4Z-KwQPKyeB6UQW1dkfdhu0rFK8=zsBz385TG6+Q@mail.gmail.com>
In-reply-to <1425657353.53.0.191129899035.issue23571@psf.upfronthosting.co.za>
Content
2015-03-06 16:55 GMT+01:00 Serhiy Storchaka <report@bugs.python.org>:
> We can ignore such little slow down (if this is not just compiler artifact).

Maybe it's time to play with micro-optimizations like __builtin_expect
:-) But I would prefer to do that in a separated issue. The patch is
already long and complex.

> The patch LGTM. I afraid there will be issues with third-party code that will need an adaptation to 3.5, but on other hand, this patch could help with catching some hidden bugs.

My patch doesn't make more strict. Without my patch, if a function
returns a result and raise an exception, the exception will still be
noticed, but probably later. The purpose of my change is to notify
immediatly that the function raised an exception.

Without my patch, if a function returns a result and raise an
exception, it's possible that the exception is cleared and so never
seen.

In older Python versions, Python/ceval.c displayed the message "XXX
undetected error" when an exception was raised but also a result was
returned.

Anyway, you must fix your code :-)
History
Date User Action Args
2015-03-06 22:22:22vstinnersetrecipients: + vstinner, serhiy.storchaka
2015-03-06 22:22:22vstinnerlinkissue23571 messages
2015-03-06 22:22:22vstinnercreate