[issue3638] tkinter.mainloop() is meaningless and crash: remove it

STINNER Victor victor.stin...@haypocalc.com added the comment: Tkapp_Mainloop is supposed to work both as a module function and a method, and it tests for self to find out which case it is. Now, this test is apparently broken in 3.x, ... Ok. In Python 2.x, selfptr is NULL whereas selfptr is

[issue3638] tkinter.mainloop() is meaningless and crash: remove it

Martin v. Löwis mar...@v.loewis.de added the comment: Ok. In Python 2.x, selfptr is NULL whereas selfptr is a pointer to the module in Python 3.x. New attached patch uses PyModule_Check() to check if selfptr is the module or an object. The patch looks right in principle. Please make sure

[issue4811] invalid reST markup in several documents

Akira Kitada akit...@gmail.com added the comment: Could you also fix the indentation of Queues example code at http://docs.python.org/library/multiprocessing.html#exchanging-objects-between-processes ? -- nosy: +akitada ___ Python tracker

[issue3638] tkinter.mainloop() is meaningless and crash: remove it

Guilherme Polo ggp...@gmail.com added the comment: Victor, you seem to be confusing code that supports it with functions that use it. There are some conditional code inside Tkapp_MainLoop that depends on self being available, that is what Martin meant by code that supports it, and since it would

[issue4818] Patch for thread-support in md5module.c

Lukas Lueg knabberknusperh...@yahoo.de added the comment: fixed naming, lock get's tried before releasing the gil to wait for it Added file: http://bugs.python.org/file12568/md5module_small_locks-2.diff ___ Python tracker rep...@bugs.python.org

[issue4818] Patch for thread-support in md5module.c

Changes by Lukas Lueg knabberknusperh...@yahoo.de: Removed file: http://bugs.python.org/file12565/md5module_small_locks.diff ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue4818 ___

[issue4820] ctypes.util.find_library incorrectly documented

New submission from David M. Beazley beaz...@users.sourceforge.net: In the ctypes reference / Finding shared libraries section of the ctypes documentation, the find_library() function is described as being located in ctypes.util. However, it's formal description right below that lists it as

[issue3638] tkinter.mainloop() is meaningless and crash: remove it

Martin v. Löwis mar...@v.loewis.de added the comment: Now, I'm much more in favour to remove it from moduleMethods than from adjusting it to work in py3k. Would you apply the same reasoning to the other Tkapp methods available on _tkinter? IIRC, these functions were there first; the Tkapp

[issue4815] idle 3.1a1 utf8

Pavel Kosina g...@post.cz added the comment: seems to be working. Seems to me now I get it. The file encoding is ruled by the encoding declaration. When I stated # -*- coding: cp1250 -*- then the file would be saved in cp1250. Now hoping that I would keep this issue, cause it comes with

[issue4815] idle 3.1a1 utf8

Martin v. Löwis mar...@v.loewis.de added the comment: Now hoping that I would keep this issue, cause it comes with this patches: when I open file *with* say # -*- coding: cp1250 -*-, I am asked to change to utf8. This behaviour was not before and is probably unwanted. Actually, the

[issue3638] tkinter.mainloop() is meaningless and crash: remove it

STINNER Victor victor.stin...@haypocalc.com added the comment: Victor, you seem to be confusing code that supports it with functions that use it Oh ok, I didn't understood what code that supports it mean. There are some conditional code inside Tkapp_MainLoop that depends on self being

[issue3638] tkinter.mainloop() is meaningless and crash: remove it

Guilherme Polo ggp...@gmail.com added the comment: Martin v. Löwis mar...@v.loewis.de added the comment: Now, I'm much more in favour to remove it from moduleMethods than from adjusting it to work in py3k. Would you apply the same reasoning to the other Tkapp methods available on

[issue4815] idle 3.1a1 utf8

Pavel Kosina g...@post.cz added the comment: Well, thanks a lot. (aware this is really off this issue): Now I even get the system of patches - issue 4008 solved the inconvenience in print Unicode signs inside IDLE. Still not sure how works patches for Python versions. I vote for including

[issue3638] tkinter.mainloop() is meaningless and crash: remove it

Martin v. Löwis mar...@v.loewis.de added the comment: But I don't see a RPC being used there, I just see some polling. Consider Tkapp_Call (e.g.). If this is invoked in the Tk interpreter thread, then there is a direct call to Tcl_EvalObjv/Tkapp_CallResult. If the call is made from a

[issue4815] idle 3.1a1 utf8

Martin v. Löwis mar...@v.loewis.de added the comment: I vote for including this a that patch about IDLE even in some 3.0.1, not only in branch 3.1. This is my plan, yes - hence I marked them all release-critical. They still need review. I agree that IDLE in 3.0 is fairly broken wrt.

[issue3638] tkinter.mainloop() is meaningless and crash: remove it

Guilherme Polo ggp...@gmail.com added the comment: Martin v. Löwis mar...@v.loewis.de added the comment: But I don't see a RPC being used there, I just see some polling. Consider Tkapp_Call (e.g.). If this is invoked in the Tk interpreter thread, then there is a direct call to

[issue3638] tkinter.mainloop() is meaningless and crash: remove it

Martin v. Löwis mar...@v.loewis.de added the comment: This is all true but the dispatching isn't used there actually. dispatching is being used in a polling manner to try to catch the thread running the tcl interpreter which someone tried to call into, the code then proceeds to do what you

[issue4821] Patches for thread-support in built-in SHA modules

New submission from Lukas Lueg knabberknusperh...@yahoo.de: Here is the follow-up to issue #4818. The patches attached allow the built-in SHA modules to release the GIL. Also the build-in SHA modules will now no longer accept s# as input. Input is parsed just as in the openssl-driven classes

[issue4751] Patch for better thread support in hashlib

Antoine Pitrou pit...@free.fr added the comment: I'm not sure about the approach of dynamically allocating self-lock. Imagine you allocate this lock while another thread is between ENTER_HASHLIB and LEAVE_HASHLIB. What happens on LEAVE_HASHLIB? The thread tries to release a lock it hadn't

[issue4821] Patches for thread-support in built-in SHA modules

Changes by Lukas Lueg knabberknusperh...@yahoo.de: -- keywords: +patch Added file: http://bugs.python.org/file12569/sha1module_small_locks.diff ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue4821

[issue4821] Patches for thread-support in built-in SHA modules

Changes by Lukas Lueg knabberknusperh...@yahoo.de: Added file: http://bugs.python.org/file12570/sha256module_small_locks.diff ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue4821 ___

[issue4821] Patches for thread-support in built-in SHA modules

Changes by Lukas Lueg knabberknusperh...@yahoo.de: Added file: http://bugs.python.org/file12571/sha512module_small_locks.diff ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue4821 ___

[issue4580] slicing of memoryviews when itemsize != 1 is wrong

Antoine Pitrou pit...@free.fr added the comment: The patch is committed in r68200, r68202. Thanks! ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue4580 ___

[issue4822] Fix indentation in memoryobject.c

New submission from Antoine Pitrou pit...@free.fr: The rules state new files should be indented with 4-spaces, but indentation has no consistent indentation: parts of it are indented with tabs, others with spaces. We could probably reindent it all with 4-spaces, especially since it hasn't been

[issue4580] slicing of memoryviews when itemsize != 1 is wrong

Changes by Antoine Pitrou pit...@free.fr: -- resolution: accepted - fixed status: open - closed ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue4580 ___

[issue4751] Patch for better thread support in hashlib

Lukas Lueg knabberknusperh...@yahoo.de added the comment: The lock is created while having the GIL in EVP_update. No other function releases the GIL (besides the creator-function which does not need the local lock). Thereby no other thread can be in between ENTER and LEAVE while the lock is

[issue4817] PyOS_GetLastModificationTime is unused

Martin v. Löwis mar...@v.loewis.de added the comment: Thanks for the patch. Committed as r68203 and r68204. -- nosy: +loewis resolution: - accepted status: open - closed ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue4817

[issue4753] Faster opcode dispatch on gcc

Changes by djc dirk...@ochtman.nl: -- nosy: +djc ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue4753 ___ ___ Python-bugs-list mailing list

[issue4822] Fix indentation in memoryobject.c

Georg Brandl ge...@python.org added the comment: I think this is the kind of reindention that Guido approved of on python-dev, along with unicodeobject.c. -- nosy: +georg.brandl ___ Python tracker rep...@bugs.python.org

[issue4822] Fix indentation in memoryobject.c

Benjamin Peterson benja...@python.org added the comment: +1 -- nosy: +benjamin.peterson ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue4822 ___ ___

[issue4823] idle height and place

New submission from Pavel Kosina g...@post.cz: Nearly always (after opening) is IDLE window outside visible area. Mainly the status bar is hidden under bottom windows menu bar. Same situation happens when choosing Window-Zoom Height from IDLE menu. xpsp3, 1024x768, py2.x-3.x --

[issue3959] Add Google's ipaddr.py to the stdlib

Duncan McGreggor duncan.mcgreg...@gmail.com added the comment: I am a contributor to netaddr, having deprecated my own old and crufty IP address library in its favor. JP's comments on the library in this ticket are included in the set of reasons that I initially chose it to replace my own.

[issue3959] Add Google's ipaddr.py to the stdlib

Martin v. Löwis mar...@v.loewis.de added the comment: Perhaps the following might be a prudent course: This sounds all good. It is better if the experts in a domain make a recommendation, than if some random committer makes a choice. As for shutting down any project that is chosen, does such

[issue4822] Fix indentation in memoryobject.c

Antoine Pitrou pit...@free.fr added the comment: Here's a patch. -- keywords: +patch Added file: http://bugs.python.org/file12572/reindent-memview.patch ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue4822

[issue3959] Add Google's ipaddr.py to the stdlib

Changes by Antoine Pitrou pit...@free.fr: -- nosy: +pitrou ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue3959 ___ ___ Python-bugs-list mailing

[issue4822] Fix indentation in memoryobject.c

Georg Brandl ge...@python.org added the comment: Do you expect a review? ;) ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue4822 ___ ___ Python-bugs-list

[issue3959] Add Google's ipaddr.py to the stdlib

Duncan McGreggor duncan.mcgreg...@gmail.com added the comment: As for shutting down any project that is chosen, does such an action not leave older Python versions out in the cold? Shouldn't the project remain open to support Python versions 2.7, with a highly visible note that the

[issue3959] Add Google's ipaddr.py to the stdlib

Antoine Pitrou pit...@free.fr added the comment: If there's going to be lots of discussion, perhaps it should be taken to python-dev? ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue3959 ___

[issue4718] wsgiref package totally broken

Antoine Pitrou pit...@free.fr added the comment: It is committed now in py3k and the 3.0 maintenance branch. Thanks all for your participation! -- resolution: - fixed status: open - closed ___ Python tracker rep...@bugs.python.org

[issue4823] idle height and place

Pavel Kosina g...@post.cz added the comment: sorry it is duplicate to issue 3286 pls close ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue4823 ___ ___

[issue3286] IDLE opens window too low on Windows

Pavel Kosina g...@post.cz added the comment: +1 -- nosy: +geon versions: +Python 3.1 ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue3286 ___ ___

[issue4796] Decimal to receive from_float method

Raymond Hettinger rhettin...@users.sourceforge.net added the comment: The direct method is *much* faster! Applied Mark's suggestions. Committed as r68208 and r68211 . -- resolution: - fixed status: open - closed versions: +Python 2.7, Python 3.1 ___

[issue4822] Fix indentation in memoryobject.c

Antoine Pitrou pit...@free.fr added the comment: Ok, committed. -- resolution: - fixed status: open - closed ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue4822 ___

[issue4823] idle height and place

Changes by Guilherme Polo ggp...@gmail.com: -- resolution: - duplicate status: open - closed ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue4823 ___

[issue3329] API for setting the memory allocator used by Python

Neil Richardson richyn...@gmail.com added the comment: I'll be in agreement here. I integrated Python into a game engine not too long ago, and had to a do a fair chunk of work to isolate Python into it's own heap - given that fragmentation on low memory systems can be a bit of a killer. Would

[issue4718] wsgiref package totally broken

Antoine Pitrou pit...@free.fr added the comment: Reopening, the patch actually produces failures when run with python -bb, that is there are comparisons between str and bytes. See the errors at the end of

[issue4718] wsgiref package totally broken

Antoine Pitrou pit...@free.fr added the comment: People, does this patch look ok to you? Added file: http://bugs.python.org/file12573/wsgiref-bb.patch ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue4718

[issue3638] tkinter.mainloop() is meaningless and crash: remove it

Guilherme Polo ggp...@gmail.com added the comment: Here is a patch, two actually. The next one deprecates the functions in trunk Added file: http://bugs.python.org/file12574/issue3638.diff ___ Python tracker rep...@bugs.python.org

[issue3638] tkinter.mainloop() is meaningless and crash: remove it

Changes by Guilherme Polo ggp...@gmail.com: Added file: http://bugs.python.org/file12575/deprecated_funcs.diff ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue3638 ___

[issue4718] wsgiref package totally broken

Dmitry Vasiliev d...@hlabs.spb.ru added the comment: Antoine Pitrou wrote: People, does this patch look ok to you? Oh, didn't know about -bb. The patch looks OK for me. ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue4718

[issue4718] wsgiref package totally broken

Antoine Pitrou pit...@free.fr added the comment: Le samedi 03 janvier 2009 à 20:24 +, Dmitry Vasiliev a écrit : Dmitry Vasiliev d...@hlabs.spb.ru added the comment: Antoine Pitrou wrote: People, does this patch look ok to you? Oh, didn't know about -bb. Well, it's meant to catch

[issue3638] tkinter.mainloop() is meaningless and crash: remove it

Martin v. Löwis mar...@v.loewis.de added the comment: These look fine. I think further changes are necessary: tkinter/__init__.py tries to load createfilehandler/deletefilehandler; this becomes redundant. Also, why did you leave DoOneEvent and Quit as-is - don't they fall into the same category?

[issue4718] wsgiref package totally broken

Dmitry Vasiliev d...@hlabs.spb.ru added the comment: Antoine Pitrou wrote: There's another problem in that buildbot failure with the environment variable NO_PROXY. We'll see if it's still there after the patch. Strange error and it seems there is only part of the traceback. I've already seen

[issue4718] wsgiref package totally broken

Antoine Pitrou pit...@free.fr added the comment: Strange error and it seems there is only part of the traceback. I've already seen such partially displayed Python 3 traceback and error actually can be in very different place. If you can reproduce such a problem, please open a bug.

[issue4815] idle 3.1a1 utf8

Pavel Kosina g...@post.cz added the comment: I might have another problem with this patch and maybe also that one in issue 4008. Having a file with print (ěščřžýáíé) # saved in cp1250 Open - confirm converting to utf8 - F5 - error: see attached file idleunicode1.jpg Added file:

[issue4718] wsgiref package totally broken

Dmitry Vasiliev d...@hlabs.spb.ru added the comment: Antoine Pitrou wrote: Strange error and it seems there is only part of the traceback. I've already seen such partially displayed Python 3 traceback and error actually can be in very different place. If you can reproduce such a problem,

[issue4815] idle 3.1a1 utf8

Martin v. Löwis mar...@v.loewis.de added the comment: print (ěščřžýáíé) # saved in cp1250 I can't reproduce the problem. Can you please attach the exact file that failed to work? ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue4815

[issue3638] tkinter.mainloop() is meaningless and crash: remove it

Guilherme Polo ggp...@gmail.com added the comment: These look fine. I think further changes are necessary: tkinter/__init__.py tries to load createfilehandler/deletefilehandler; this becomes redundant. Indeed, I forgot to look into the Python code. I also see as redundant the checks for

[issue3638] tkinter.mainloop() is meaningless and crash: remove it

Guilherme Polo ggp...@gmail.com added the comment: I get an empty string with the text below, but I was expecting some s/text/test ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue3638 ___

[issue4815] idle 3.1a1 utf8

Pavel Kosina g...@post.cz added the comment: Martin v. Löwis napsal(a), dne 3.1.2009 22:24: I can't reproduce the problem. Can you please attach the exact file that failed to work? You can use that one that is already here: cp1250.py. It is the same error with me.

[issue3638] tkinter.mainloop() is meaningless and crash: remove it

Martin v. Löwis mar...@v.loewis.de added the comment: But these all would deserve another(s) issues, so I will be moving quit and dooneevent from there too. I haven't tried reproducing these problems, but this all sounds plausible. So go ahead and check in all the changes for this issue.

[issue4815] idle 3.1a1 utf8

Martin v. Löwis mar...@v.loewis.de added the comment: You can use that one that is already here: cp1250.py. It is the same error with me. Ok, then what are the exact steps to reproduce? What code base, what patches applied, what user interaction in what order?

[issue4815] idle 3.1a1 utf8

Pavel Kosina g...@post.cz added the comment: Microsoft Windows XP [Verze 5.1.2600] (C) Copyright 1985-2001 Microsoft Corp. C:\prg\Python30\Lib\idlelibsvn update# from http://svn.python.org/projects/python/branches/py3k/Lib/idlelib Restored 'AutoCompleteWindow.py' Restored 'ToolTip.py'

[issue4718] wsgiref package totally broken

Dmitry Vasiliev d...@hlabs.spb.ru added the comment: Attached patch for test_urllib, possible source of the NO_PROXY problem. Added file: http://bugs.python.org/file12577/no_proxy.patch ___ Python tracker rep...@bugs.python.org

[issue4815] idle 3.1a1 utf8

Martin v. Löwis mar...@v.loewis.de added the comment: I see. Here is a revised patch. This sets the modified flag on the buffer after conversion, so that you get asked to save it before running it. Added file: http://bugs.python.org/file12578/conv.diff ___

[issue3638] Remove module level functions in _tkinter that depend on TkappObject

Guilherme Polo ggp...@gmail.com added the comment: Thanks Martin, trunk: r68231 (blocked on py3k) py3k: r68237 release30-maint: r68239 -- resolution: - fixed status: open - closed title: tkinter.mainloop() is meaningless and crash: remove it - Remove module level functions in

[issue4753] Faster opcode dispatch on gcc

Changes by Yann Ramin at...@stackworks.net: -- nosy: +theatrus ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue4753 ___ ___ Python-bugs-list mailing

[issue4815] idle 3.1a1 utf8

Pavel Kosina g...@post.cz added the comment: Yes. God job. ;-) ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue4815 ___ ___ Python-bugs-list mailing

[issue3638] Remove module level functions in _tkinter that depend on TkappObject

Guilherme Polo ggp...@gmail.com added the comment: Uh, I forgot about the code removal in __init__ in the first commits. r68242, r68244 now ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue3638 ___

[issue2983] Ttk support for Tkinter

Brett Cannon br...@python.org added the comment: Do you still need a review for this, Guilherme? If you just need a quick code check I can see if I can make the time for this, but my tkinter knowledge is lacking so it won't be very in-depth. Hopefully this is all isolated enough that even if it

[issue2983] Ttk support for Tkinter

Guilherme Polo ggp...@gmail.com added the comment: Do you still need a review for this, Guilherme? If you just need a quick code check I can see if I can make the time for this, but my tkinter knowledge is lacking so it won't be very in-depth. Hopefully this is all isolated enough that even

[issue4718] wsgiref package totally broken

Antoine Pitrou pit...@free.fr added the comment: Nice catch! I've committed the two patches and we'll see whether it makes the buildbots feel better. -- resolution: - accepted stage: patch review - committed/rejected status: open - pending ___

[issue2983] Ttk support for Tkinter

Brett Cannon br...@python.org added the comment: If Fredrik already reviewed it I don't see why you can't commit. If something goes wrong you can always back the commit out. As for the testing question, I am not sure I understand what you are asking. Are you asking if the tests can stay

[issue2983] Ttk support for Tkinter

Guilherme Polo ggp...@gmail.com added the comment: As for the testing question, I am not sure I understand what you are asking. Are you asking if the tests can stay integrated with tkinter's existing tests? There are no tests for tkinter, except for some few ones for _tkinter basic

[issue4824] test_cmd_line failure on Mac OS X for py3k

New submission from Skip Montanaro s...@pobox.com: test_cmd_line.test_run_code fails for me on Mac OS X: % ./python.exe -bb Lib/test/regrtest.py -v test_cmd_line test_cmd_line test_directories (test.test_cmd_line.CmdLineTest) ... ok test_optimize

[issue4825] TypeError with complex.real() and complex.imag()

New submission from MagnetoHydroDynamics k...@asmussen.ws: In both version 2.6.1 and 3.0 this issue exists: N and M can be two numbers of any type. (N+Mj).real() Traceback (most recent call last): File pyshell#1, line 1, in module (N+Mj).real() TypeError: 'float' object is not callable

[issue4825] TypeError with complex.real() and complex.imag()

MagnetoHydroDynamics k...@asmussen.ws added the comment: In both version 2.6.1 and 3.0 this issue exists: N and M can be two numbers of any type. (N+Mj).real() Traceback (most recent call last): File pyshell#1, line 1, in module (N+Mj).real() TypeError: 'float' object is not callable

[issue1702551] distutils sdist does not exclude SVN/CVS files on Windows

Georg Brandl ge...@python.org added the comment: Tarek, I don't know if you are already subscribed to the python-checkins mailing list -- I've reviewed the commit and posted a reply with a minor problem there. -- nosy: +georg.brandl ___ Python

[issue4825] TypeError with complex.real() and complex.imag()

Pierre Bourdon r...@delroth.is-a-geek.org added the comment: I don't think this is a valid issue : real and imag are just properties of complex objects, not methods ! -- nosy: +delroth ___ Python tracker rep...@bugs.python.org

[issue4753] Faster opcode dispatch on gcc

Changes by Benoit Boissinot bboissin+pythonb...@gmail.com: -- nosy: +bboissin ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue4753 ___ ___

[issue4825] TypeError with complex.real() and complex.imag()

MagnetoHydroDynamics k...@asmussen.ws added the comment: Nvm. I thought it was metods. Requesting deletion. ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue4825 ___

[issue1702551] distutils sdist does not exclude SVN/CVS files on Windows

Tarek Ziadé ziade.ta...@gmail.com added the comment: Georg, I think I am supposed to be registered since a few days but I don't receive any mail yet. I'll ask... I didn't use a raw string because '\.' is not an escape sequence, so: '\.svn' == '\\.svn' and '\.svn' == r'\.svn' True

[issue4824] test_cmd_line failure on Mac OS X for py3k

Skip Montanaro s...@pobox.com added the comment: Here is another data point. I added some print calls to the subprocess module and ran the key call from the interpreter: % ./python.exe Python 3.1a0 (py3k:68218, Jan 3 2009, 15:06:30) [GCC 4.0.1 (Apple Inc. build 5490)] on darwin

[issue3997] zipfile and winzip

Changes by Martin v. Löwis mar...@v.loewis.de: -- versions: +Python 2.7, Python 3.0, Python 3.1 ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue3997 ___

[issue4753] Faster opcode dispatch on gcc

Daniel Diniz aja...@gmail.com added the comment: IIUC, this is what gcc 4.2.4 generates on a Celeron M for the code Alexandre posted: movl-272(%ebp), %eax movl8(%ebp), %edx subl-228(%ebp), %eax movl%eax, 60(%edx) movl-272(%ebp), %ecx

[issue4753] Faster opcode dispatch on gcc

Paolo 'Blaisorblade' Giarrusso p.giarru...@gmail.com added the comment: 1st note: is that code from the threaded version? Note that you need to modify the source to make it accept also ICC to try that. In case you already did that, I guess the patch is not useful at all with ICC since, as far as

[issue4753] Faster opcode dispatch on gcc

Paolo 'Blaisorblade' Giarrusso p.giarru...@gmail.com added the comment: Daniel, I forgot to ask for the compilation command line you used, since they make a lot of difference. Can you post them? Thanks ___ Python tracker rep...@bugs.python.org

[issue4826] exec() docstring bug about file objects

New submission from xverify xver...@yahoo.com: exec() no longer accepts a file object but its docstring states otherwise. -- assignee: georg.brandl components: Documentation messages: 79036 nosy: georg.brandl, xverify severity: normal status: open title: exec() docstring bug about file

[issue4753] Faster opcode dispatch on gcc

Alexandre Vassalotti alexan...@peadrop.com added the comment: Paolo wrote: So, can you try dropping the switch altogether, using always computed goto and seeing how does the resulting code get compiled? Removing the switch won't be possible unless we change the semantic EXTENDED_ARG. In

[issue4827] optparse: Callback example 1 is confusing

New submission from Jason Kankiewicz jkankiew...@acm.org: Callback example 1: trivial callback reads Here’s an example of a callback option that takes no arguments, and simply records that the option was seen: def record_foo_seen(option, opt_str, value, parser):

<    1   2