On Sat, Jul 21, 2012 at 5:41 PM, Georg Brandl <g.bra...@gmx.net> wrote:
> This looks like it will give an "error return without exception set"
> if the "if (expat_capi)" directly before the change is true, and
> expat_capi is then set to NULL in the branch.
>
> Georg
>

Good catch! I'll fix it soon.
Eli



> On 07/17/2012 01:25 PM, eli.bendersky wrote:
>> http://hg.python.org/cpython/rev/d896fd0a8ba7
>> changeset:   78152:d896fd0a8ba7
>> branch:      3.2
>> parent:      78142:71537aba3a0a
>> user:        Eli Bendersky <eli...@gmail.com>
>> date:        Tue Jul 17 14:20:38 2012 +0300
>> summary:
>>   ISsue #14988: restore Python 2's behavior of raising ImportError when 
>> unable to load pyexpat, instead of a SystemError/RuntimeError
>>
>> files:
>>   Modules/_elementtree.c |  2 ++
>>   1 files changed, 2 insertions(+), 0 deletions(-)
>>
>>
>> diff --git a/Modules/_elementtree.c b/Modules/_elementtree.c
>> --- a/Modules/_elementtree.c
>> +++ b/Modules/_elementtree.c
>> @@ -3117,6 +3117,8 @@
>>              expat_capi->MICRO_VERSION != XML_MICRO_VERSION)
>>              expat_capi = NULL;
>>      }
>> +    if (!expat_capi)
>> +        return NULL;
>>  #endif
>>
>>      elementtree_parseerror_obj = PyErr_NewException(
>>
>>
>>
>> _______________________________________________
>> Python-checkins mailing list
>> python-check...@python.org
>> http://mail.python.org/mailman/listinfo/python-checkins
>>
>
>
> _______________________________________________
> 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/eliben%40gmail.com
_______________________________________________
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

Reply via email to