From: Raymond Hettinger <python@rcn.com>
Date: Wed, 5 Dec 2007 18:11:08 +0000 (+0000)
Subject: Error checking was too aggressive (reported by Chris Tismer)
X-Git-Tag: v2.6a1~928
X-Git-Url: https://granicus.if.org/sourcecode?a=commitdiff_plain;h=2e1af256d475543a1a6c0c8c54cb2497560600bc;p=python

Error checking was too aggressive (reported by Chris Tismer)
---

diff --git a/Lib/collections.py b/Lib/collections.py
index 356d961203..e551f20d8e 100644
--- a/Lib/collections.py
+++ b/Lib/collections.py
@@ -40,7 +40,7 @@ def namedtuple(typename, field_names, verbose=False):
         field_names = field_names.replace(',', ' ').split() # names separated by whitespace and/or commas
     field_names = tuple(field_names)
     for name in (typename,) + field_names:
-        if not name.replace('_', '').isalnum():
+        if not all(c.isalnum() or c=='_' for c in name):
             raise ValueError('Type names and field names can only contain alphanumeric characters and underscores: %r' % name)
         if _iskeyword(name):
             raise ValueError('Type names and field names cannot be a keyword: %r' % name)
@@ -48,7 +48,7 @@ def namedtuple(typename, field_names, verbose=False):
             raise ValueError('Type names and field names cannot start with a number: %r' % name)
     seen_names = set()
     for name in field_names:
-        if name.startswith('__') and name.endswith('__'):
+        if name.startswith('__') and name.endswith('__') and len(name) > 3:
             raise ValueError('Field names cannot start and end with double underscores: %r' % name)
         if name in seen_names:
             raise ValueError('Encountered duplicate field name: %r' % name)
diff --git a/Lib/test/test_collections.py b/Lib/test/test_collections.py
index 52bae9afed..0b0b029d7f 100644
--- a/Lib/test/test_collections.py
+++ b/Lib/test/test_collections.py
@@ -29,6 +29,7 @@ class TestNamedTuple(unittest.TestCase):
         self.assertRaises(ValueError, namedtuple, 'abc', 'efg efg ghi')    # duplicate field
 
         namedtuple('Point0', 'x1 y2')   # Verify that numbers are allowed in names
+        namedtuple('_', '_ __ ___')     # Verify that underscores are allowed
 
     def test_instance(self):
         Point = namedtuple('Point', 'x y')