[Python-checkins] r51994 - in python/branches/release25-maint: Lib/test/test_future.py Misc/NEWS Parser/parser.c
georg.brandl
python-checkins at python.org
Sun Sep 24 14:35:41 CEST 2006
Author: georg.brandl
Date: Sun Sep 24 14:35:40 2006
New Revision: 51994
Modified:
python/branches/release25-maint/Lib/test/test_future.py
python/branches/release25-maint/Misc/NEWS
python/branches/release25-maint/Parser/parser.c
Log:
Fix a bug in the parser's future statement handling that led to "with"
not being recognized as a keyword after, e.g., this statement:
from __future__ import division, with_statement
(backport from rev. 51993)
Modified: python/branches/release25-maint/Lib/test/test_future.py
==============================================================================
--- python/branches/release25-maint/Lib/test/test_future.py (original)
+++ python/branches/release25-maint/Lib/test/test_future.py Sun Sep 24 14:35:40 2006
@@ -82,6 +82,23 @@
else:
self.fail("expected exception didn't occur")
+ def test_parserhack(self):
+ # test that the parser.c::future_hack function works as expected
+ try:
+ exec "from __future__ import division, with_statement; with = 0"
+ except SyntaxError:
+ pass
+ else:
+ self.fail("syntax error didn't occur")
+
+ try:
+ exec "from __future__ import (with_statement, division); with = 0"
+ except SyntaxError:
+ pass
+ else:
+ self.fail("syntax error didn't occur")
+
+
def test_main():
test_support.run_unittest(FutureTest)
Modified: python/branches/release25-maint/Misc/NEWS
==============================================================================
--- python/branches/release25-maint/Misc/NEWS (original)
+++ python/branches/release25-maint/Misc/NEWS Sun Sep 24 14:35:40 2006
@@ -12,6 +12,10 @@
Core and builtins
-----------------
+- Fix a bug in the parser's future statement handling that led to "with"
+ not being recognized as a keyword after, e.g., this statement:
+ from __future__ import division, with_statement
+
- Allow exception instances to be directly sliced again.
Modified: python/branches/release25-maint/Parser/parser.c
==============================================================================
--- python/branches/release25-maint/Parser/parser.c (original)
+++ python/branches/release25-maint/Parser/parser.c Sun Sep 24 14:35:40 2006
@@ -181,7 +181,7 @@
future_hack(parser_state *ps)
{
node *n = ps->p_stack.s_top->s_parent;
- node *ch;
+ node *ch, *cch;
int i;
/* from __future__ import ..., must have at least 4 children */
@@ -195,15 +195,17 @@
if (NCH(ch) == 1 && STR(CHILD(ch, 0)) &&
strcmp(STR(CHILD(ch, 0)), "__future__") != 0)
return;
- for (i = 3; i < NCH(n); i += 2) {
- /* XXX: assume we don't have parentheses in import:
- from __future__ import (x, y, z)
- */
- ch = CHILD(n, i);
- if (NCH(ch) == 1)
- ch = CHILD(ch, 0);
- if (NCH(ch) >= 1 && TYPE(CHILD(ch, 0)) == NAME &&
- strcmp(STR(CHILD(ch, 0)), "with_statement") == 0) {
+ ch = CHILD(n, 3);
+ /* ch can be a star, a parenthesis or import_as_names */
+ if (TYPE(ch) == STAR)
+ return;
+ if (TYPE(ch) == LPAR)
+ ch = CHILD(n, 4);
+
+ for (i = 0; i < NCH(ch); i += 2) {
+ cch = CHILD(ch, i);
+ if (NCH(cch) >= 1 && TYPE(CHILD(cch, 0)) == NAME &&
+ strcmp(STR(CHILD(cch, 0)), "with_statement") == 0) {
ps->p_flags |= CO_FUTURE_WITH_STATEMENT;
break;
}
More information about the Python-checkins
mailing list