Issue #6620: Slightly safer code for _grouping_intervals in the locale

module.  Fixes a 'possible use before assignment' warning from pylint.
Thanks Vincent Legoll.

diff --git a/Lib/locale.py b/Lib/locale.py
index 879725feb94a72df85e67175b109f23bd3f96013..f0733999d2c3a587948cf937ac28c1048d73ed38 100644 (file)
--- a/Lib/locale.py
+++ b/Lib/locale.py
@@ -114,12 +114,15 @@ def localeconv():
 
 # Iterate over grouping intervals
 def _grouping_intervals(grouping):
+    last_interval = None
     for interval in grouping:
         # if grouping is -1, we are done
         if interval == CHAR_MAX:
             return
         # 0: re-use last group ad infinitum
         if interval == 0:
+            if last_interval is None:
+                raise ValueError("invalid grouping")
             while True:
                 yield last_interval
         yield interval