[Python-checkins] commit of r41767 - in python/trunk: Misc/NEWS Python/symtable.c

[neal.norwitz]

Author: neal.norwitz
Date: Mon Dec 19 05:27:42 2005
New Revision: 41767

Modified:
   python/trunk/Misc/NEWS
   python/trunk/Python/symtable.c
Log:
Bug #889500, fix line number on SyntaxWarning for global declarations.



Modified: python/trunk/Misc/NEWS
==============================================================================
--- python/trunk/Misc/NEWS	(original)
+++ python/trunk/Misc/NEWS	Mon Dec 19 05:27:42 2005
@@ -12,6 +12,8 @@
 Core and builtins
 -----------------
 
+- Bug #889500, fix line number on SyntaxWarning for global declarations.
+
 - Bug #1378022, UTF-8 files with a leading BOM crashed the interpreter.
 
 - Support for converting hex strings to floats no longer works.

Modified: python/trunk/Python/symtable.c
==============================================================================
--- python/trunk/Python/symtable.c	(original)
+++ python/trunk/Python/symtable.c	Mon Dec 19 05:27:42 2005
@@ -4,13 +4,16 @@
 #include "symtable.h"
 #include "structmember.h"
 
-/* two error strings used for warnings */
+/* error strings used for warnings */
 #define GLOBAL_AFTER_ASSIGN \
 "name '%.400s' is assigned to before global declaration"
 
 #define GLOBAL_AFTER_USE \
 "name '%.400s' is used prior to global declaration"
 
+#define IMPORT_STAR_WARNING "import * only allowed at module level"
+
+
 PySTEntryObject *
 PySTEntry_New(struct symtable *st, identifier name, _Py_block_ty block,
 	      void *key, int lineno)
@@ -152,7 +155,7 @@
 };
 
 static int symtable_analyze(struct symtable *st);
-static int symtable_warn(struct symtable *st, char *msg);
+static int symtable_warn(struct symtable *st, char *msg, int lineno);
 static int symtable_enter_block(struct symtable *st, identifier name, 
 				_Py_block_ty block, void *ast, int lineno);
 static int symtable_exit_block(struct symtable *st, void *ast);
@@ -686,10 +689,10 @@
 
 
 static int
-symtable_warn(struct symtable *st, char *msg)
+symtable_warn(struct symtable *st, char *msg, int lineno)
 {
 	if (PyErr_WarnExplicit(PyExc_SyntaxWarning, msg, st->st_filename,
-			       st->st_cur->ste_lineno, NULL, NULL) < 0)	{
+			       lineno, NULL, NULL) < 0)	{
 		if (PyErr_ExceptionMatches(PyExc_SyntaxWarning)) {
 			PyErr_SetString(PyExc_SyntaxError, msg);
 			PyErr_SyntaxLocation(st->st_filename, 
@@ -1028,7 +1031,7 @@
 					PyOS_snprintf(buf, sizeof(buf),
 						      GLOBAL_AFTER_USE,
 						      c_name);
-				if (!symtable_warn(st, buf))
+				if (!symtable_warn(st, buf, s->lineno))
                                     return 0;
 			}
 			if (!symtable_add_def(st, name, DEF_GLOBAL))
@@ -1277,8 +1280,8 @@
 	}
 	else {
             if (st->st_cur->ste_type != ModuleBlock) {
-                if (!symtable_warn(st,
-                                   "import * only allowed at module level")) {
+                int lineno = st->st_cur->ste_lineno;
+                if (!symtable_warn(st, IMPORT_STAR_WARNING, lineno)) {
                     Py_DECREF(store_name);
                     return 0;
 		}