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 brett.cannon
Recipients ag6502, alex, benjamin.peterson, brett.cannon, eric.araujo, ezio.melotti, kermit666
Date 2013-07-06.17:22:44
SpamBayes Score -1.0
Marked as misclassified Yes
Message-id <1373131364.98.0.0896627070693.issue18156@psf.upfronthosting.co.za>
In-reply-to
Content
Dražen: didn't do a deep review, but a cursory look suggests the patch is fine.

As for having to change a ton of files to start using the attribute, that's part of the effect of changing something as basic as an exception. If you would rather break it up into separate patches that's fine. But I would caution you from doing more than this patch as I have gotten some push back from other core devs on my proposed new attributes on exceptions so I don't want you spending your time on something that might get sunk (although if you are enjoying it then go for it since if it gets accepted this work will be needed).

Also please sign the contributor agreement form (http://python.org/psf/contrib/contrib-form/) so that we can actually use your code.

Andrea: While the attribute might coincidentally name an attribute that is the same as some other attribute which is not the actual trigger, the traceback on the exception provides the proper context to know what attribute in what code did the actual triggering. The point is that the exception can be considered better than nothing and is still an improvement over not having the information at all.
History
Date User Action Args
2013-07-06 17:22:45brett.cannonsetrecipients: + brett.cannon, ag6502, benjamin.peterson, ezio.melotti, eric.araujo, alex, kermit666
2013-07-06 17:22:44brett.cannonsetmessageid: <1373131364.98.0.0896627070693.issue18156@psf.upfronthosting.co.za>
2013-07-06 17:22:44brett.cannonlinkissue18156 messages
2013-07-06 17:22:44brett.cannoncreate