RetroSearch Browse

Tue Jul 10 17:34:18 CEST 2007 · http://mail.python.org/pipermail/python-checkins/2007-July/061036.html

Author: collin.winter
Date: Tue Jul 10 17:34:18 2007
New Revision: 56242

Modified:
   sandbox/trunk/2to3/pytree.py
   sandbox/trunk/2to3/tests/test_pytree.py
Log:
Fix a bug where set_prefix() didn't register that the node had changed.

Modified: sandbox/trunk/2to3/pytree.py
==============================================================================

--- sandbox/trunk/2to3/pytree.py	(original)
+++ sandbox/trunk/2to3/pytree.py	Tue Jul 10 17:34:18 2007
@@ -304,6 +304,7 @@
 
     def set_prefix(self, prefix):
         """Sets the prefix for the node."""
+        self.changed()
         self.prefix = prefix
 
     def get_prefix(self):

Modified: sandbox/trunk/2to3/tests/test_pytree.py
==============================================================================
--- sandbox/trunk/2to3/tests/test_pytree.py	(original)
+++ sandbox/trunk/2to3/tests/test_pytree.py	Tue Jul 10 17:34:18 2007
@@ -67,8 +67,10 @@
     def testLeafPrefix(self):
         l1 = pytree.Leaf(100, "foo")
         self.assertEqual(l1.get_prefix(), "")
+        self.failIf(l1.was_changed)
         l1.set_prefix("  ##\n\n")
         self.assertEqual(l1.get_prefix(), "  ##\n\n")
+        self.failUnless(l1.was_changed)
 
     def testNode(self):
         l1 = pytree.Leaf(100, "foo")


Home - News ( United States | United Kingdom | Italy | Germany ) - Football scores

Showing content from http://mail.python.org/pipermail/python-checkins/2007-July/061036.html below:

[Python-checkins] r56242 - in sandbox/trunk/2to3: pytree.py tests/test_pytree.py