On Wed, Feb 1, 2012 at 3:07 PM, Ethan Furman <et...@stoneleaf.us> wrote: > I'm looking at the docs to make the relevant changes due to PEP 409, and I'm > noticing some problems. > > E.g. The PyException_Get|Set_Context|Cause all talk about using NULL to > clear the related attribute, when actually in should be Py_None. > > Only PyException_GetCause is directly related to PEP 409 -- should I only > fix that one, and open up a new issue on the tracker for the other three, or > should I fix all four now?
Passing in NULL is the right way to clear them using those APIs - the descriptors in exceptions.c then control how "not set" is exposed at the Python layer. So only Get/SetCause should need updating for PEP 409 to say to pass in NULL to clear the cause and fall back on displaying the context and Py_None to suppress the context in the default display. Cheers, Nick. -- Nick Coghlan | ncogh...@gmail.com | Brisbane, Australia _______________________________________________ Python-Dev mailing list Python-Dev@python.org http://mail.python.org/mailman/listinfo/python-dev Unsubscribe: http://mail.python.org/mailman/options/python-dev/archive%40mail-archive.com
