[Python-3000-checkins] r58987 - in python/branches/py3k: Lib/test/test_parser.py Modules/parsermodule.c
Brett Cannon
brett at python.org
Fri Nov 16 01:32:36 CET 2007
On Nov 15, 2007 11:35 AM, Guido van Rossum <guido at python.org> wrote:
> Sorry. There's one real change:
>
> - validate_testlist(CHILD(tree, 3)) &&
> + validate_arglist(CHILD(tree, 3)) &&
>
OK, so I am blind. =)
-Brett
>
> On Nov 15, 2007 11:31 AM, Brett Cannon <brett at python.org> wrote:
> > Am I going blind, or is there only whitespace changes to
> > parsermodule.c? If so, is there supposed to be a change there? Or is
> > the test the big deal?
> >
> > -Brett
> >
> > On Nov 15, 2007 11:17 AM, guido.van.rossum
> >
> > <python-3000-checkins at python.org> wrote:
> > > Author: guido.van.rossum
> > > Date: Thu Nov 15 20:17:28 2007
> > > New Revision: 58987
> > >
> > > Modified:
> > > python/branches/py3k/Lib/test/test_parser.py
> > > python/branches/py3k/Modules/parsermodule.c
> > > Log:
> > > Patch 1144 by David Binger, fix for parser module. With unittest.
> > > (I also cleared out all trailing whitespace in the C file.)
> > >
> > >
> > > Modified: python/branches/py3k/Lib/test/test_parser.py
> > > ==============================================================================
> > > --- python/branches/py3k/Lib/test/test_parser.py (original)
> > > +++ python/branches/py3k/Lib/test/test_parser.py Thu Nov 15 20:17:28 2007
> > > @@ -136,6 +136,7 @@
> > >
> > > def test_class_defs(self):
> > > self.check_suite("class foo():pass")
> > > + self.check_suite("class foo(object):pass")
> > >
> > > def test_import_from_statement(self):
> > > self.check_suite("from sys.path import *")
> > >
> > > Modified: python/branches/py3k/Modules/parsermodule.c
> > > ==============================================================================
> > > --- python/branches/py3k/Modules/parsermodule.c (original)
> > > +++ python/branches/py3k/Modules/parsermodule.c Thu Nov 15 20:17:28 2007
> > > @@ -861,7 +861,7 @@
> > > VALIDATER(class); VALIDATER(node);
> > > VALIDATER(parameters); VALIDATER(suite);
> > > VALIDATER(testlist); VALIDATER(varargslist);
> > > -VALIDATER(vfpdef);
> > > +VALIDATER(vfpdef);
> > > VALIDATER(stmt); VALIDATER(simple_stmt);
> > > VALIDATER(expr_stmt); VALIDATER(power);
> > > VALIDATER(del_stmt);
> > > @@ -874,7 +874,7 @@
> > > VALIDATER(try); VALIDATER(except_clause);
> > > VALIDATER(test); VALIDATER(and_test);
> > > VALIDATER(not_test); VALIDATER(comparison);
> > > -VALIDATER(comp_op);
> > > +VALIDATER(comp_op);
> > > VALIDATER(star_expr); VALIDATER(expr);
> > > VALIDATER(xor_expr); VALIDATER(and_expr);
> > > VALIDATER(shift_expr); VALIDATER(arith_expr);
> > > @@ -988,11 +988,11 @@
> > > else {
> > > (void) validate_numnodes(tree, 4, "class");
> > > }
> > > -
> > > +
> > > if (res) {
> > > if (nch == 7) {
> > > res = ((validate_lparen(CHILD(tree, 2)) &&
> > > - validate_testlist(CHILD(tree, 3)) &&
> > > + validate_arglist(CHILD(tree, 3)) &&
> > > validate_rparen(CHILD(tree, 4))));
> > > }
> > > else if (nch == 6) {
> > > @@ -1177,11 +1177,11 @@
> > > }
> > > while (res && i+1 < nch) { /* validate (',' vfpdef ['=' test])* */
> > > res = validate_comma(CHILD(tree, i));
> > > - if (TYPE(CHILD(tree, i+1)) == DOUBLESTAR)
> > > + if (TYPE(CHILD(tree, i+1)) == DOUBLESTAR)
> > > break;
> > > res = res && validate_vfpdef(CHILD(tree, i+1));
> > > if (res && i+2 < nch && TYPE(CHILD(tree, i+2)) == EQUAL) {
> > > - res = res && (i+3 < nch)
> > > + res = res && (i+3 < nch)
> > > && validate_test(CHILD(tree, i+3));
> > > i += 4;
> > > }
> > > @@ -1234,7 +1234,7 @@
> > > int sym;
> > > node *ch;
> > > int i = 0;
> > > -
> > > +
> > > if (!res)
> > > return 0;
> > > if (nch < 1) {
> > > @@ -1242,7 +1242,7 @@
> > > return 0;
> > > }
> > > while (i < nch) {
> > > - ch = CHILD(tree, i);
> > > + ch = CHILD(tree, i);
> > > sym = TYPE(ch);
> > > if (sym == vfpdef || sym == tfpdef) {
> > > /* validate (vfpdef ['=' test] ',')+ */
> > > @@ -1443,7 +1443,7 @@
> > > static int
> > > validate_yield_or_testlist(node *tree)
> > > {
> > > - if (TYPE(tree) == yield_expr)
> > > + if (TYPE(tree) == yield_expr)
> > > return validate_yield_expr(tree);
> > > else
> > > return validate_testlist(tree);
> > > @@ -1675,7 +1675,7 @@
> > > && validate_dotted_as_names(CHILD(tree, 1)));
> > > }
> > >
> > > -/* Helper function to count the number of leading dots in
> > > +/* Helper function to count the number of leading dots in
> > > * 'from ...module import name'
> > > */
> > > static int
> > > @@ -2361,7 +2361,7 @@
> > > static int
> > > validate_decorators(node *tree)
> > > {
> > > - int i, nch, ok;
> > > + int i, nch, ok;
> > > nch = NCH(tree);
> > > ok = validate_ntype(tree, decorators) && nch >= 1;
> > >
> > > @@ -2372,7 +2372,7 @@
> > > }
> > >
> > > /* funcdef:
> > > - *
> > > + *
> > > * -5 -4 -3 -2 -1
> > > * 'def' NAME parameters ':' suite
> > > */
> > > _______________________________________________
> > > Python-3000-checkins mailing list
> > > Python-3000-checkins at python.org
> > > http://mail.python.org/mailman/listinfo/python-3000-checkins
> > >
> > _______________________________________________
> > Python-3000-checkins mailing list
> > Python-3000-checkins at python.org
> > http://mail.python.org/mailman/listinfo/python-3000-checkins
> >
>
>
>
> --
> --Guido van Rossum (home page: http://www.python.org/~guido/)
>
More information about the Python-3000-checkins
mailing list