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