RetroSearch Browse

Wed Apr 18 15:39:34 CEST 2012 · https://mail.python.org/pipermail/python-dev/2012-April/118844.html

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

Home - News ( United States | United Kingdom | Italy | Germany ) - Football scores

Showing content from https://mail.python.org/pipermail/python-dev/2012-April/118844.html below:

Fix email post-commit review comments.