compiler.parse("d[1] = 1") should have a single tuple as subs


>>> compiler.parse("d[1] = 1")
Module(None, Stmt([Assign([Subscript(Name('d'), 'OP_ASSIGN',
[Const(1)])], Const(1))]))
>>> compiler.parse("d[1,] = 2")
Module(None, Stmt([Assign([Subscript(Name('d'), 'OP_ASSIGN',
[Const(1)])], Const(2))]))
>>> compiler.parse("d[1,2] = 3")
Module(None, Stmt([Assign([Subscript(Name('d'), 'OP_ASSIGN', [Const(1),
Const(2)])], Const(3))]))
>>> compiler.parse("d[(1,)] = 2")
Module(None, Stmt([Assign([Subscript(Name('d'), 'OP_ASSIGN',
[Tuple([Const(1)])])], Const(2))]))

I just see that my patch is not correct, since the following is
supported by the language:

O[1:2:3, 4:5:6]

Where O[1:2:3, 4:5:6] == O[slice(1,2,3), slice(4,5,6)] ==
O.__getitem__((slice(1,2,3), slice(4,5,6)))

patch which honors O[1:2:3, 4:5:6]

your patch is no more a diff file (the 'previous' file is empty)

Sorry. Renamed .bak to .orig ...
Here's the patch compiler/transformer.py (against python 2.5)

PythonTidy encounters this problem.

o http://lacusveris.com/PythonTidy/PythonTidy.python

It is unable correctly to render line 694 of test_grammar.py in the
Python Test Suite:

    d[1,] = 2

becomes:

    d[1] = 2

because the *compiler* module does not return an abstract syntax tree
containing a tuple for the subscript.

The patch should have a test.