[Patches] [ python-Patches-1548388 ] set comprehensions
SourceForge.net
noreply at sourceforge.net
Wed Aug 30 00:30:22 CEST 2006
Patches item #1548388, was opened at 2006-08-29 03:33
Message generated for change (Comment added) made by rhettinger
You can respond by visiting:
https://sourceforge.net/tracker/?func=detail&atid=305470&aid=1548388&group_id=5470
Please note that this message will contain a full copy of the comment thread,
including the initial issue submission, for this request,
not just the latest update.
Category: Core (C code)
Group: Python 3000
Status: Open
Resolution: None
Priority: 5
Submitted By: Georg Brandl (gbrandl)
Assigned to: Guido van Rossum (gvanrossum)
Summary: set comprehensions
Initial Comment:
This is a big one:
* cleanup grammar; unifies listcomp/genexp grammar
which means that [x for x in 1, 2] is no longer valid
* cleanup comprehension compiling code (unifies all AST
code for the three comprehensions and most of the
compile.c code)
* add set comprehensions
This patch modifies list comprehensions to be
implemented more like generator expressions: in a
separate function, which means that the loop variables
will not leak any more.
----------------------------------------------------------------------
>Comment By: Raymond Hettinger (rhettinger)
Date: 2006-08-29 17:30
Message:
Logged In: YES
user_id=80475
Can you post a before and disassembly of some list and set
comprehensions.
----------------------------------------------------------------------
Comment By: Georg Brandl (gbrandl)
Date: 2006-08-29 14:09
Message:
Logged In: YES
user_id=849994
test_compiler and test_transformer fail because the compiler
package hasn't been updated yet.
test_dis fails because list comprehensions now generate
different bytecode.
----------------------------------------------------------------------
Comment By: Guido van Rossum (gvanrossum)
Date: 2006-08-29 12:59
Message:
Logged In: YES
user_id=6380
Nice!
I see failures in 4 tests:
test_compiler test_dis test_transformer test_univnewlines
test_univnewlines is trivial (it's deleting a variable
leaked out of a list comprehension); haven't looked at the
rest in detail
----------------------------------------------------------------------
Comment By: Georg Brandl (gbrandl)
Date: 2006-08-29 03:34
Message:
Logged In: YES
user_id=849994
The previously attached patch contains only the important
files. The FULL patch (attached now) also contains syntax
fixes in python files so that the test suite is mostly passing.
Note that the compiler package isn't ready yet.
----------------------------------------------------------------------
You can respond by visiting:
https://sourceforge.net/tracker/?func=detail&atid=305470&aid=1548388&group_id=5470
More information about the Patches
mailing list