Treat def f(a, b=1, c): ... as an error (missing default for c)

instead of silently supplying a default of None fore c.

Treat def f(a, b=1, c): ... as an error (missing default for c)
instead of silently supplying a default of None fore c.
29d38cd0 · Guido van Rossum · b7c61318 · 29d38cd0
Commit 29d38cd0 authored Oct 02, 1998 by Guido van Rossum
Hide whitespace changes
Inline Side-by-side

Showing with 4 additions and 7 deletions

Python/compile.c Python/compile.c +4 -7

No files found.
--- a/Python/compile.c
+++ b/Python/compile.c
@@ -2721,13 +2721,10 @@ com_argdefs(c, n)
 			t = TYPE(CHILD(n, i));
 		}
 		else {
-			/* Treat "(a=1, b)" as "(a=1, b=None)" */
+			/* Treat "(a=1, b)" as an error */
-			if (ndefs) {
+			if (ndefs)
-				com_addoparg(c, LOAD_CONST,
+				com_error(c, PyExc_SyntaxError,
-					     com_addconst(c, Py_None));
+					  "Missing parameter default value");
-				com_push(c, 1);
-				ndefs++;
-			}
 		}
 		if (t != COMMA)
 			break;