[Patches] [ python-Patches-737999 ] minor codeop fixes

[SourceForge.net]

Patches item #737999, was opened at 2003-05-15 02:55
Message generated for change (Comment added) made by mwh
You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=305470&aid=737999&group_id=5470

Category: Library (Lib)
Group: None
Status: Open
>Resolution: Accepted
Priority: 5
Submitted By: Samuele Pedroni (pedronis)
>Assigned to: Samuele Pedroni (pedronis)
Summary: minor codeop fixes

Initial Comment:
*) compile_command("",eval) -> None ,
previously raised syntax error about "pass"

*) symbol must be eval or single, now
raises ValueError accordingly as specified in the 
documentation

patch both for 2.3 and 2.2 maint for codeop.py and 
test/test_codeop.py (beefed up)

(for Jython at the moment we use the test suite of 2.2 
maint, that's why this is relevant)

----------------------------------------------------------------------

>Comment By: Michael Hudson (mwh)
Date: 2003-05-16 13:00

Message:
Logged In: YES 
user_id=6656

Given that codeop has *never* lived up to its promise to
raise ValueError on 'exec', perhaps the docs should be fixed
rather than the code?  (I don't understand the point of
supporting anything but 'single', but...).  Certainly this
change *should not* be made on the 22-maint branch.

Can you check in your tests to both branches and port
Guido's codeop.py change?  Fixing up the docs gets you extra
points, I think  :-)

----------------------------------------------------------------------

You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=305470&aid=737999&group_id=5470