Merged revisions 67030-67031 via svnmerge from

benjaminp · benjaminp · commit 9aebc61ad8e3 · 2008-10-26T20:58:53.000Z
svn+ssh://pythondev@svn.python.org/python/trunk

........
  r67030 | benjamin.peterson | 2008-10-26 15:21:13 -0500 (Sun, 26 Oct 2008) | 1 line

  fix __future__ imports when multiple features are given
........
  r67031 | benjamin.peterson | 2008-10-26 15:33:19 -0500 (Sun, 26 Oct 2008) | 1 line

  add forgotten test for r67030
........
diff --git a/Lib/test/test_future.py b/Lib/test/test_future.py
@@ -89,19 +89,23 @@ def test_parserhack(self):
         #       the parser hack disabled. If a new keyword is introduced in
         #       2.6, change this to refer to the new future import.
         try:
-            exec("from __future__ import division, with_statement; with = 0")
+            exec("from __future__ import print_function; print 0")
         except SyntaxError:
             pass
         else:
             self.fail("syntax error didn't occur")
 
         try:
-            exec("from __future__ import (with_statement, division); with = 0")
+            exec("from __future__ import (print_function); print 0")
         except SyntaxError:
             pass
         else:
             self.fail("syntax error didn't occur")
 
+    def test_multiple_features(self):
+        support.unload("test.test_future5")
+        from test import test_future5
+
 
 def test_main():
     support.run_unittest(FutureTest)
diff --git a/Lib/test/test_future5.py b/Lib/test/test_future5.py
@@ -0,0 +1,21 @@
+# Check that multiple features can be enabled.
+from __future__ import unicode_literals, print_function
+
+import sys
+import unittest
+from . import support
+
+
+class TestMultipleFeatures(unittest.TestCase):
+
+    def test_unicode_literals(self):
+        self.assertTrue(isinstance("", str))
+
+    def test_print_function(self):
+        with support.captured_output("stderr") as s:
+            print("foo", file=sys.stderr)
+        self.assertEqual(s.getvalue(), "foo\n")
+
+
+def test_main():
+    support.run_unittest(TestMultipleFeatures)
diff --git a/Parser/parser.c b/Parser/parser.c
@@ -210,13 +210,10 @@ future_hack(parser_state *ps)
 			char *str_ch = STR(CHILD(cch, 0));
 			if (strcmp(str_ch, FUTURE_WITH_STATEMENT) == 0) {
 				ps->p_flags |= CO_FUTURE_WITH_STATEMENT;
-				break;
 			} else if (strcmp(str_ch, FUTURE_PRINT_FUNCTION) == 0) {
 				ps->p_flags |= CO_FUTURE_PRINT_FUNCTION;
-				break;
 			} else if (strcmp(str_ch, FUTURE_UNICODE_LITERALS) == 0) {
 				ps->p_flags |= CO_FUTURE_UNICODE_LITERALS;
-				break;
 			}
 		}
 	}

Original file line number	Diff line number	Diff line change
`@@ -210,13 +210,10 @@ future_hack(parser_state *ps)`
`210`	`210`	`char *str_ch = STR(CHILD(cch, 0));`
`211`	`211`	`if (strcmp(str_ch, FUTURE_WITH_STATEMENT) == 0) {`
`212`	`212`	`ps->p_flags \|= CO_FUTURE_WITH_STATEMENT;`
`213`		`- break;`
`214`	`213`	`} else if (strcmp(str_ch, FUTURE_PRINT_FUNCTION) == 0) {`
`215`	`214`	`ps->p_flags \|= CO_FUTURE_PRINT_FUNCTION;`
`216`		`- break;`
`217`	`215`	`} else if (strcmp(str_ch, FUTURE_UNICODE_LITERALS) == 0) {`
`218`	`216`	`ps->p_flags \|= CO_FUTURE_UNICODE_LITERALS;`
`219`		`- break;`
`220`	`217`	`}`
`221`	`218`	`}`
`222`	`219`	`}`