[Python-Dev] [Python-checkins] cpython: Fix email post-commit review comments.

R. David Murray rdmurray at bitdance.com
Wed Apr 18 16:08:10 CEST 2012


We're seeing segfuilts on the buildbots now.  Example:

http://www.python.org/dev/buildbot/all/builders/x86%20Ubuntu%20Shared%203.x/builds/5715

On Wed, 18 Apr 2012 23:39:34 +1000, Nick Coghlan <ncoghlan at gmail.com> wrote:
> On Wed, Apr 18, 2012 at 11:31 PM, brian.curtin
> <python-checkins at python.org> wrote:
> > -    if (name == NULL)
> > +    if (name == NULL) {
> > +        Py_INCREF(Py_None);
> >         name = Py_None;
> > +    }
> 
> A slightly more traditional way to write that would be:
> 
>     name = Py_None;
>     Py_INCREF(name);
> 
> > -    if (path == NULL)
> > +    if (path == NULL) {
> > +        Py_INCREF(Py_None);
> >         path = Py_None;
> > +    }
> 
> Ditto.
> 
> >
> >     Py_INCREF(msg);
> > -    PyTuple_SetItem(args, 0, msg);
> > +    PyTuple_SetItem(args, 0, NULL);//msg);
> 
> However, *this* looks a lot more suspicious... accidental commit of
> debugging code?
> 
> (if not for spotting this last problem, I wouldn't have even mentioned
> the first two)
> 
> Cheers,
> Nick.
> 
> -- 
> Nick Coghlan   |   ncoghlan at gmail.com   |   Brisbane, Australia
> _______________________________________________
> Python-Dev mailing list
> Python-Dev at python.org
> http://mail.python.org/mailman/listinfo/python-dev
> Unsubscribe: http://mail.python.org/mailman/options/python-dev/rdmurray%40bitdance.com


More information about the Python-Dev mailing list