[issue3323] Clarify __slots__ behaviour when inheriting

2008-07-09 Thread Andy
New submission from Andy [EMAIL PROTECTED]: Suggest clarification on behaviour of the __slots__ attribute when inheriting from classes that don't have __slots__ defined. Obviously the superclass automatically creates __dict__, and it seems the subclass inherits this. I presume this is expected

[issue3238] backport python 3.0 language functionality to python 2.6 by adding 7 opcodes to ceval.c

2008-07-09 Thread kai zhu
kai zhu [EMAIL PROTECTED] added the comment: update: these 3k language features have been tested to work in python 2.5.2 w/ the backported opcodes pep3104 Access to Names in Outer Scopes pep3105 Make print a function pep3111 Simple input built-in in Python 3000 pep3113 Removal

[issue3324] Broken link in online doc

2008-07-09 Thread ThomasH
New submission from ThomasH [EMAIL PROTECTED]: The page http://docs.python.org/inst/search-path.html contains a broken link to site module documentation. Is: http://www.python.org/doc/devel/lib/module-site.html Should be: http://docs.python.org/lib/module-site.html -- assignee:

[issue3325] use of cPickle in multiprocessing

2008-07-09 Thread Andrii V. Mishkovskyi
New submission from Andrii V. Mishkovskyi [EMAIL PROTECTED]: There are two places in multiprocessing where cPickle (gone from py3k already) is used. Both of them are in try .. except, so they don't break code. Here is a patch that removes these uses. -- components: Library (Lib)

[issue3325] use of cPickle in multiprocessing

2008-07-09 Thread Andrii V. Mishkovskyi
Andrii V. Mishkovskyi [EMAIL PROTECTED] added the comment: And here is the patch. -- keywords: +patch Added file: http://bugs.python.org/file10867/issue3325.diff ___ Python tracker [EMAIL PROTECTED] http://bugs.python.org/issue3325

[issue3326] py3k shouldn't use -fno-strict-aliasing anymore

2008-07-09 Thread Ismail Donmez
New submission from Ismail Donmez [EMAIL PROTECTED]: py3k branch is still using -fno-strict-aliasing but I tested with gcc 4.3.1 and there are no strict aliasing warnings when this flag is removed. Attached patch for py3k branch removes this flag. After applying the patch configure should be

[issue874900] threading module can deadlock after fork

2008-07-09 Thread Jesse Noller
Jesse Noller [EMAIL PROTECTED] added the comment: Amaury - your latest patch doesn't seem to apply cleanly to trunk, it's choking on the Python/ceval.c changes ___ Python tracker [EMAIL PROTECTED] http://bugs.python.org/issue874900

[issue874900] threading module can deadlock after fork

2008-07-09 Thread Amaury Forgeot d'Arc
Amaury Forgeot d'Arc [EMAIL PROTECTED] added the comment: Here is a third patch with latest trunk. Did you apply it to a clean checkout? Added file: http://bugs.python.org/file10869/fork-and-thread3.patch ___ Python tracker [EMAIL PROTECTED]

[issue874900] threading module can deadlock after fork

2008-07-09 Thread Jesse Noller
Jesse Noller [EMAIL PROTECTED] added the comment: I had to flip on ignore whitespace to patch: patch -p0 -l ~/Desktop/fork-and-thread3.patch Worked. Unfortunately, test_threading is locking up during a make test. Here's the verbose regrtest.py output: woot:python-trunk jesse$ ./python.exe

[issue3327] NULL member in modules_by_index

2008-07-09 Thread Kristján Valur Jónsson
New submission from Kristján Valur Jónsson [EMAIL PROTECTED]: In _PyState_AddModule(), a list of (initially) 20 elements is created, but there is no guarantee that all elements are initialized. In particular, it appears that element 0 is always NULL. This is bad since this list is

[issue874900] threading module can deadlock after fork

2008-07-09 Thread Jesse Noller
Jesse Noller [EMAIL PROTECTED] added the comment: Amaury, it looks like it's hanging in subprocess: /Users/jesse/open_source/subversion/python- trunk/Lib/test/test_threading.py(338)_run_and_join() - p = subprocess.Popen([sys.executable, -c, script], stdout=subprocess.PIPE) (Pdb) step

[issue1758146] Crash in PyObject_Malloc

2008-07-09 Thread Franco DiRosa
Franco DiRosa [EMAIL PROTECTED] added the comment: OK, I think I found my problem. I was using the main interpreter state (the one created by Py_Initialize) to create new thread states with. It seems that this interpreter state is reserved for GIL functions so one will need to create a new

[issue3300] urllib.quote and unquote - Unicode issues

2008-07-09 Thread Tom Pinckney
Tom Pinckney [EMAIL PROTECTED] added the comment: I mentioned this is in a brief python-dev discussion earlier this spring, but many popular websites such as Wikipedia and Facebook do use UTF-8 as their character encoding scheme for the path and argument portion of URLs. I know there's no

[issue3300] urllib.quote and unquote - Unicode issues

2008-07-09 Thread Matt Giuca
Matt Giuca [EMAIL PROTECTED] added the comment: OK I've gone back over the patch and decided to add the encoding and errors arguments from the str.encode/decode methods as optional arguments to quote and unquote. This is a much bigger change than I originally intended, but I think it makes

[issue3328] When PyObject_CallMethod fails, refcount is incorrect

2008-07-09 Thread Dominic Lavoie
New submission from Dominic Lavoie [EMAIL PROTECTED]: This issue is similar to issue 1229429. In the called Python method, a regular expression fails to compile PyEval_CallMethod() returns 0. However, the reference count is incremented by 1 which prevents the object from being destroyed.

[issue3328] When PyObject_CallMethod fails, refcount is incorrect

2008-07-09 Thread Amaury Forgeot d'Arc
Amaury Forgeot d'Arc [EMAIL PROTECTED] added the comment: No, PyTuple_SET_ITEM() steals a reference to its argument, so that ownership is transferred to the tuple. The reference will be released when the tuple is disposed, with Py_DECREF(arg). What makes you think that there is a reference

[issue874900] threading module can deadlock after fork

2008-07-09 Thread Jesse Noller
Jesse Noller [EMAIL PROTECTED] added the comment: Here's the good news, with the patch applied to trunk, I'm not seeing hangs in the multiprocessing test suite. I'm running it on a tight loop excluding the threading tests to confirm. ___ Python tracker

[issue874900] threading module can deadlock after fork

2008-07-09 Thread Amaury Forgeot d'Arc
Amaury Forgeot d'Arc [EMAIL PROTECTED] added the comment: Well I was a bit presumptuous that my thread+fork tests would pass on all platforms out of the box. We should disable the tests, or have someone with better Unix expertise examine and correct them. At least I feel that the actual

[issue874900] threading module can deadlock after fork

2008-07-09 Thread Adam Olsen
Adam Olsen [EMAIL PROTECTED] added the comment: In general I suggest replacing the lock with a new lock, rather than trying to release the existing one. Releasing *might* work in this case, only because it's really a semaphore underneath, but it's still easier to think about by just replacing.

[issue874900] threading module can deadlock after fork

2008-07-09 Thread Adam Olsen
Adam Olsen [EMAIL PROTECTED] added the comment: Looking over some of the other platforms for thread_*.h, I'm sure replacing the lock is the right thing. ___ Python tracker [EMAIL PROTECTED] http://bugs.python.org/issue874900

[issue3328] When PyObject_CallMethod fails, refcount is incorrect

2008-07-09 Thread Dominic Lavoie
Dominic Lavoie [EMAIL PROTECTED] added the comment: My application is fairly complex so I am currently trying to build a simple test case that reproduces the problem. Will let you know as soon as it is ready. Regards, Dominic ___ Python tracker [EMAIL

[issue3328] When PyObject_CallMethod fails, refcount is incorrect

2008-07-09 Thread Dominic Lavoie
Dominic Lavoie [EMAIL PROTECTED] added the comment: OK, I have been able to reproduce the problem with a simple test program. All you have to do is compile issue328.c and run it. issue328.py compiles an invalid regexp. You will see that the refcount goes from 1 to 2 if the regexp compilation

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

2008-07-09 Thread Jukka Laurila
New submission from Jukka Laurila [EMAIL PROTECTED]: Currently Python always uses the C library malloc/realloc/free as the underlying mechanism for requesting memory from the OS, but especially on memory-limited platforms it is often desirable to be able to override the allocator and to redirect

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

2008-07-09 Thread Brett Cannon
Brett Cannon [EMAIL PROTECTED] added the comment: Is registering pointers to functions really necessary, or would defining macros work as well? From a performance perspective I would like to avoid having a pointer indirection step every time malloc/realloc/free is called. I guess my question

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

2008-07-09 Thread Adam Olsen
Adam Olsen [EMAIL PROTECTED] added the comment: How would this allow you to free all memory? The interpreter will still reference it, so you'd have to have called Py_Finalize already, and promise not to call Py_Initialize afterwords. This further supposes the process will live a long time

[issue3300] urllib.quote and unquote - Unicode issues

2008-07-09 Thread Martin v. Löwis
Changes by Martin v. Löwis [EMAIL PROTECTED]: -- versions: +Python 3.1 -Python 3.0 ___ Python tracker [EMAIL PROTECTED] http://bugs.python.org/issue3300 ___ ___

[issue3300] urllib.quote and unquote - Unicode issues

2008-07-09 Thread Martin v. Löwis
Martin v. Löwis [EMAIL PROTECTED] added the comment: Assuming the patch is acceptable in the first place (which I personally have not made my mind up), then it lacks documentation and test suite changes. ___ Python tracker [EMAIL PROTECTED]

[issue2542] PyErr_ExceptionMatches must not fail

2008-07-09 Thread Guido van Rossum
Guido van Rossum [EMAIL PROTECTED] added the comment: FWIW some comments by Amaury are here: http://codereview.appspot.com/483 -- nosy: +gvanrossum ___ Python tracker [EMAIL PROTECTED] http://bugs.python.org/issue2542

[issue2303] isinstance is 4x as slow as in 2.5 because the common case raises

2008-07-09 Thread Guido van Rossum
Guido van Rossum [EMAIL PROTECTED] added the comment: Does anyone care about this still? I added some comments on http://codereview.appspot.com/504 ___ Python tracker [EMAIL PROTECTED] http://bugs.python.org/issue2303 ___

[issue3330] webbrowser module doesn't correctly handle '|' character.

2008-07-09 Thread Adrian Petrescu
New submission from Adrian Petrescu [EMAIL PROTECTED]: The webbrowser module seems to treat URLs containing the | character differently based on whether the browser is already running or not. For instance, consider the following python script: import webbrowser url =

[issue3330] webbrowser module doesn't correctly handle '|' character.

2008-07-09 Thread Adrian Petrescu
Adrian Petrescu [EMAIL PROTECTED] added the comment: Just as an aside, the reason I consider this a fairly serious bug is that the Google Charts API urls make heavy use of the '|' character, which means if I want to have Python use it by opening the user's browser, it won't work if they don't

[issue3328] When PyObject_CallMethod fails, refcount is incorrect

2008-07-09 Thread Amaury Forgeot d'Arc
Amaury Forgeot d'Arc [EMAIL PROTECTED] added the comment: Thanks for the short case. It makes everything simpler. First, note that you get the same behaviour with python-only code: import re, sys class Issue3328: def f(self): # unbalanced parenthesis x = re.compile('(123')

[issue1662581] the re module can perform poorly: O(2**n) versus O(n**2)

2008-07-09 Thread Yarko Tymciurak
Yarko Tymciurak [EMAIL PROTECTED] added the comment: Not sure if this is a real-world case of this in particular, but possibly: http://groups.google.com/group/web2py/browse_thread/thread/59ff2e31698bced6/9bbae2d482d11b88 -- nosy: +yarkot ___ Python

[issue874900] threading module can deadlock after fork

2008-07-09 Thread Amaury Forgeot d'Arc
Amaury Forgeot d'Arc [EMAIL PROTECTED] added the comment: A new patch: - I replaced _active_limbo_lock.release() by _active_limbo_lock=_allocate_lock() - I replaced the successive deletions in _active by a re-creation with only the current thread. There is no difference in the result, but I

[issue3300] urllib.quote and unquote - Unicode issues

2008-07-09 Thread Matt Giuca
Matt Giuca [EMAIL PROTECTED] added the comment: OK well here are the necessary changes to the documentation (RST docs and docstrings in the code). As I said above, I plan to to extensive testing and add new cases, and I don't recommend this patch is accepted until that's done. Patch