Add warnings for arguments named None. All set. (I could add a

warning for 'global None', but that's either accompanied by an
assignment to None, which will trigger a warning, or not, in which
case it's harmless. :-)

diff --git a/Python/compile.c b/Python/compile.c
index 0edbc528d3d0be7b4ebbf19d771c10746a45a70c..0109fe539c8e6a4f3eda4c6b8caa1407a0366945 100644 (file)
--- a/Python/compile.c
+++ b/Python/compile.c
@@ -5025,6 +5025,14 @@ symtable_add_def(struct symtable *st, char *name, int flag)
        char buffer[MANGLE_LEN];
        int ret;
 
+       /* Warn about None, except inside a tuple (where the assignment
+          code already issues a warning). */
+       if ((flag & DEF_PARAM) && !(flag & DEF_INTUPLE) &&
+           *name == 'N' && strcmp(name, "None") == 0)
+       {
+               if (symtable_warn(st, "argument named None"))
+                       return -1;
+       }
        if (_Py_Mangle(st->st_private, name, buffer, sizeof(buffer)))
                name = buffer;
        if ((s = PyString_InternFromString(name)) == NULL)
@@ -5310,7 +5318,7 @@ symtable_funcdef(struct symtable *st, node *n)
 }
 
 /* The next two functions parse the argument tuple.
-   symtable_default_arg() checks for names in the default arguments,
+   symtable_default_args() checks for names in the default arguments,
    which are references in the defining scope.  symtable_params()
    parses the parameter names, which are defined in the function's
    body.