From: Christian Heimes <christian@python.org>
Date: Fri, 23 Sep 2016 18:26:30 +0000 (+0200)
Subject: Issue #28100: Refactor error messages, patch by Ivan Levkivskyi
X-Git-Tag: v3.6.0b2~131
X-Git-Url: https://granicus.if.org/sourcecode?a=commitdiff_plain;h=517507c6d5d740ab548faddb9e0c57092d19a188;p=python

Issue #28100: Refactor error messages, patch by Ivan Levkivskyi
---

diff --git a/Python/symtable.c b/Python/symtable.c
index f762904240..bf30eccdab 100644
--- a/Python/symtable.c
+++ b/Python/symtable.c
@@ -1282,15 +1282,13 @@ symtable_visit_stmt(struct symtable *st, stmt_ty s)
                 VISIT_QUIT(st, 0);
             if (cur & (DEF_LOCAL | USE | DEF_ANNOT)) {
                 char* msg;
-                if (cur & DEF_ANNOT) {
+                if (cur & USE) {
+                    msg = GLOBAL_AFTER_USE;
+                } else if (cur & DEF_ANNOT) {
                     msg = GLOBAL_ANNOT;
-                }
-                if (cur & DEF_LOCAL) {
+                } else {  /* DEF_LOCAL */
                     msg = GLOBAL_AFTER_ASSIGN;
                 }
-                else {
-                    msg = GLOBAL_AFTER_USE;
-                }
                 PyErr_Format(PyExc_SyntaxError,
                              msg, name);
                 PyErr_SyntaxLocationObject(st->st_filename,
@@ -1315,15 +1313,13 @@ symtable_visit_stmt(struct symtable *st, stmt_ty s)
                 VISIT_QUIT(st, 0);
             if (cur & (DEF_LOCAL | USE | DEF_ANNOT)) {
                 char* msg;
-                if (cur & DEF_ANNOT) {
+                if (cur & USE) {
+                    msg = NONLOCAL_AFTER_USE;
+                } else if (cur & DEF_ANNOT) {
                     msg = NONLOCAL_ANNOT;
-                }
-                if (cur & DEF_LOCAL) {
+                } else {  /* DEF_LOCAL */
                     msg = NONLOCAL_AFTER_ASSIGN;
                 }
-                else {
-                    msg = NONLOCAL_AFTER_USE;
-                }
                 PyErr_Format(PyExc_SyntaxError, msg, name);
                 PyErr_SyntaxLocationObject(st->st_filename,
                                            s->lineno,