[issue15667] PEP 3121, 384 refactoring applied to pickle module

2013-12-01 Thread Alexandre Vassalotti
Alexandre Vassalotti added the comment: I have implemented PEP 3121 module finalization for _pickle in 64c6d52793be. I don't see the use case for implementing PEP 384 stable ABI, since _pickle is only distributed with Python. -- assignee: - alexandre.vassalotti nosy:

[issue4727] copyreg doesn't support keyword only arguments in __new__

2013-12-01 Thread Alexandre Vassalotti
Alexandre Vassalotti added the comment: PEP 3154 implemented support for pickling classes taking keyword-only arguments. The copy module should be updated to use __getnewargs_ex__ when available through object.__reduce__(4). -- nosy: +alexandre.vassalotti superseder: - Implement PEP

[issue4727] copyreg doesn't support keyword only arguments in __new__

2013-12-01 Thread Alexandre Vassalotti
Changes by Alexandre Vassalotti alexan...@peadrop.com: -- superseder: Implement PEP 3154 (pickle protocol 4) - ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue4727 ___

[issue10717] Multiprocessing module Pickling unPickling issues

2013-12-01 Thread Alexandre Vassalotti
Alexandre Vassalotti added the comment: Without a reproducible test case, I am afraid there is nothing we can do here. -- assignee: - alexandre.vassalotti components: +Library (Lib) -None nosy: +alexandre.vassalotti resolution: - works for me stage: - test needed status: open -

[issue7105] weak dict iterators are fragile because of unpredictable GC runs

2013-12-01 Thread Kristján Valur Jónsson
Kristján Valur Jónsson added the comment: Here's a different approach. Simply avoid the use of iterators over the underlying container. Instead, we iterate over lists of items/keys/values etc. -- Added file: http://bugs.python.org/file32932/weakref.patch

[issue19837] Wire protocol encoding for the JSON module

2013-12-01 Thread Nick Coghlan
Nick Coghlan added the comment: The parallel API would have to be: json.dump_bytes json.dumps_bytes json.load_bytes json.loads_bytes That is hardly an improvement over: json.bytes.dump json.bytes.dumps json.bytes.load json.bytes.loads It doesn't need to be documented as a completely separate

[issue10717] Multiprocessing module Pickling unPickling issues

2013-12-01 Thread Alexandre Vassalotti
Alexandre Vassalotti added the comment: Look like I isolated the problem. It seems multiprocessing is using cPickle which cannot be extended with ForkingPickler, unlike the Python version of the pickle module. 15:09:29 [ ~/pythondev/python2.7 ]$ ./python.exe issue10717.py Traceback (most

[issue19837] Wire protocol encoding for the JSON module

2013-12-01 Thread Antoine Pitrou
Antoine Pitrou added the comment: The parallel API would have to be: json.dump_bytes json.dumps_bytes json.load_bytes json.loads_bytes No, only one function dump_bytes() is needed, and it would return a bytes object (dumps meaning dump string, already). loads() can be polymorphic without

[issue10717] Multiprocessing module cannot call instance methods across processes

2013-12-01 Thread Alexandre Vassalotti
Changes by Alexandre Vassalotti alexan...@peadrop.com: -- title: Multiprocessing module Pickling unPickling issues - Multiprocessing module cannot call instance methods across processes ___ Python tracker rep...@bugs.python.org

[issue15798] subprocess.Popen() fails if 0, 1 or 2 descriptor is closed

2013-12-01 Thread Roundup Robot
Roundup Robot added the comment: New changeset 2df5e1f537b0 by Gregory P. Smith in branch 'default': Fixes issue #15798: subprocess.Popen() no longer fails if file http://hg.python.org/cpython/rev/2df5e1f537b0 -- ___ Python tracker

[issue7105] weak dict iterators are fragile because of unpredictable GC runs

2013-12-01 Thread Guido van Rossum
Guido van Rossum added the comment: I appreciate the simplicity, but I don't think it is acceptable -- if the dict is large, materializing the entire list of keys/values/items might allocate a prohibitive amount of memory. It's worse if you have code that is expected to break out of the loop.

[issue19754] pickletools.optimize doesn't reframe correctly

2013-12-01 Thread Alexandre Vassalotti
Alexandre Vassalotti added the comment: Now, pickletools.optimize doesn't do anything on protocol 4. :) -- assignee: - alexandre.vassalotti resolution: - fixed stage: - committed/rejected status: open - closed ___ Python tracker

[issue19754] pickletools.optimize doesn't reframe correctly

2013-12-01 Thread Roundup Robot
Roundup Robot added the comment: New changeset bb71baa28f1b by Alexandre Vassalotti in branch 'default': Issue #19754: Make pickletools.optimize respect the frame size target. http://hg.python.org/cpython/rev/bb71baa28f1b -- nosy: +python-dev ___

[issue19754] pickletools.optimize doesn't reframe correctly

2013-12-01 Thread Roundup Robot
Roundup Robot added the comment: New changeset 9feada79a411 by Alexandre Vassalotti in branch 'default': Issue #19754: Fix typo. http://hg.python.org/cpython/rev/9feada79a411 -- ___ Python tracker rep...@bugs.python.org

[issue19780] Pickle 4 frame headers optimization

2013-12-01 Thread Alexandre Vassalotti
Alexandre Vassalotti added the comment: Yeah, let's close this. It is much simpler to just double the frame size target if the extra reads ever become a performance issue. -- status: pending - closed ___ Python tracker rep...@bugs.python.org

[issue19858] Make pickletools.optimize aware of the MEMOIZE opcode.

2013-12-01 Thread Alexandre Vassalotti
New submission from Alexandre Vassalotti: PEP 3154 introduced the MEMOIZE opcode which lowered the overhead of memoization compared to the PUT opcodes which were previously used. We should update pickletools.optimize to remove superfluous uses of this new opcode. -- components:

[issue19834] Unpickling exceptions pickled by Python 2

2013-12-01 Thread Alexandre Vassalotti
Alexandre Vassalotti added the comment: Feel free to commit once you have addressed the remaining comments. -- assignee: - doerwalter ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue19834 ___

[issue18073] pickle.Unpickler may read too many bytes, causing hangs with blocking input stream

2013-12-01 Thread Alexandre Vassalotti
Changes by Alexandre Vassalotti alexan...@peadrop.com: -- assignee: - alexandre.vassalotti resolution: - fixed stage: - committed/rejected status: open - closed ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue18073

[issue19835] Add a MemoryError singleton to fix an unlimited loop when the memory is exhausted

2013-12-01 Thread Alexandre Vassalotti
Changes by Alexandre Vassalotti alexan...@peadrop.com: -- nosy: -alexandre.vassalotti ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue19835 ___ ___

[issue15798] subprocess.Popen() fails if 0, 1 or 2 descriptor is closed

2013-12-01 Thread Roundup Robot
Roundup Robot added the comment: New changeset 07425df887b5 by Gregory P. Smith in branch '3.3': Fixes issue #15798: subprocess.Popen() no longer fails if file http://hg.python.org/cpython/rev/07425df887b5 -- ___ Python tracker rep...@bugs.python.org

[issue19509] No SSL match_hostname() in ftp, imap, nntp, pop, smtp modules

2013-12-01 Thread Roundup Robot
Roundup Robot added the comment: New changeset aa531135bc6b by Christian Heimes in branch 'default': Issue #19509: Add SSLContext.check_hostname to match the peer's certificate http://hg.python.org/cpython/rev/aa531135bc6b -- nosy: +python-dev ___

[issue19781] No SSL match_hostname() in ftplib

2013-12-01 Thread Roundup Robot
Roundup Robot added the comment: New changeset 373797990f57 by Christian Heimes in branch 'default': Issue #19781: ftplib now supports SSLContext.check_hostname and server name http://hg.python.org/cpython/rev/373797990f57 -- nosy: +python-dev ___

[issue17909] Autodetecting JSON encoding

2013-12-01 Thread Julian Berman
Changes by Julian Berman julian+python@grayvines.com: -- nosy: +Julian ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue17909 ___ ___

[issue16074] Bad error message in os.rename, os.link, and os.symlink

2013-12-01 Thread Vajrasky Kok
Changes by Vajrasky Kok sky@speaklikeaking.com: -- title: bad error message in os.rename - Bad error message in os.rename, os.link, and os.symlink ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue16074

[issue15798] subprocess.Popen() fails if 0, 1 or 2 descriptor is closed

2013-12-01 Thread Gregory P. Smith
Gregory P. Smith added the comment: i went with the less invasive in terms of behavior change approach of making sure that the errpipe_write fd is always = 3. In Python 3.4 the code change was different and much simpler and on the Python only side as all fd's are opened O_CLOEXEC by default.

[issue16074] Bad error message in os.rename, os.link, and os.symlink

2013-12-01 Thread Vajrasky Kok
Vajrasky Kok added the comment: Here is the patch for Python 3.4. It removed file information for os.rename, os.link, and os.symlink. I agree with Ezio that giving extra information is better but since Python 3.4 is in beta maybe we can defer this to Python 3.5. There are couples of

[issue16074] Bad error message in os.rename, os.link, and os.symlink

2013-12-01 Thread Vajrasky Kok
Vajrasky Kok added the comment: Patch for Python 3.3. It omitted file information for error messages in rename, symlink, and link. I'll create a separate ticket for adding extra file information to these error messages. -- Added file:

<    1   2