[issue28503] [Patch] '_crypt' module: fix implicit declaration of crypt(), use crypt_r() where available

2016-10-28 Thread Xavier de Gaye
Changes by Xavier de Gaye : -- nosy: +xdegaye ___ Python tracker ___ ___ Python-bugs-list

[issue28549] curses: calling addch() with an 1-length str segfaults with ncurses6 compiled with --enable-ext-colors

2016-10-28 Thread Yutao Yuan
New submission from Yutao Yuan: When addch() is called with an 1-length str, it is converted into a cchar_t. Ncurses6 adds a new field ext_color to cchar_t if it is enabled at compile time, and it is not initialized here, which causes various problems like segfaults or wrong display of

[issue25152] venv documentation doesn't tell you how to specify a particular version of python

2016-10-28 Thread Zachary Ware
Zachary Ware added the comment: Ah ha, I thought I'd seen it mentioned in an issue somewhere, but I didn't go looking when I noticed my Docs bots were all red. Thanks for the patch anyway, Mariatta! -- nosy: +zach.ware ___ Python tracker

[issue28546] Better explain setting pdb breakpoints

2016-10-28 Thread Xavier de Gaye
Changes by Xavier de Gaye : -- nosy: +xdegaye ___ Python tracker ___ ___ Python-bugs-list

[issue28550] if inline statement does not work with multiple assignment.

2016-10-28 Thread Eric Snow
Eric Snow added the comment: The syntax is working correctly. Precedence rules are throwing you off. It should be more clear if we use parentheses to demonstrate precedence explicitly. You expected: a, b = (obj.a, obj.b) if obj else [None, None] However, what you wrote is equivalent to:

[issue28550] if inline statement does not work with multiple assignment.

2016-10-28 Thread Levi Velázquez
New submission from Levi Velázquez: Normal "if" inline assignment obj = None a = obj.a if obj else None It assign None to a as expected. obj = None a, b = obj.a, obj.b if obj else [None, None] It raises an error " 'NoneType' object has no attribute 'a'" when it should assign None to both a

[issue28522] can't make IDLEX work with python._pth and python-3.6.0b2

2016-10-28 Thread Steve Dower
Steve Dower added the comment: "Lib\site-packages" is probably unnecessary because of "import site", which likely adds it in anyway. It's very likely that WinPython doesn't actually want to specify this at all, since it also enables isolated mode, which will ignore PYTHONPATH and the current

[issue26862] android: SYS_getdents64 does not need to be defined on android API 21

2016-10-28 Thread cschramm
cschramm added the comment: Well, the 3.5 code checks __ANDROID__ as well and works pretty fine on Android, but if it's not supposed to be supported, we'll have to upgrade to 3.6 then. Thanks for your work! :) -- ___ Python tracker

[issue27683] ipaddress subnet slicing iterator malfunction

2016-10-28 Thread Xiang Zhang
Xiang Zhang added the comment: Ping. ;-) -- nosy: +pmoody ___ Python tracker ___ ___ Python-bugs-list mailing

[issue2943] Distutils should generate a better error message when the SDK is not installed

2016-10-28 Thread Ivan Pozdeev
Ivan Pozdeev added the comment: What's wrong with the Python wiki link in msg263206 above? It is supposed to be _the_ guide for setting up compilers in Windows, isn't it? Ideally, this should be a link to `distutils' docs - because, you know, the wiki isn't carved in stone, either, while the

[issue25152] venv documentation doesn't tell you how to specify a particular version of python

2016-10-28 Thread Brett Cannon
Brett Cannon added the comment: It looks like Zachary beat me to fixing this: https://hg.python.org/cpython/rev/5d1934c27137. Thanks for the patch, Mariatta, even if Zachary didn't use it. -- stage: commit review -> resolved status: open -> closed

[issue28544] Implement asyncio.Task in C

2016-10-28 Thread Yury Selivanov
Yury Selivanov added the comment: I've committed the patch with a few touchups: * Applied code-review feedback. * We now use AC for all methods, including Task._repr_info, Task._step, etc. See [1]. * I fixed the Task to be fully subclassable; users can override Task._step and Task._wakeup

[issue28544] Implement asyncio.Task in C

2016-10-28 Thread Roundup Robot
Roundup Robot added the comment: New changeset db5ae4f6df8a by Yury Selivanov in branch '3.6': Issue #28544: Implement asyncio.Task in C. https://hg.python.org/cpython/rev/db5ae4f6df8a New changeset 8059289b55c1 by Yury Selivanov in branch 'default': Merge 3.6 (issue #28544)

[issue28544] Implement asyncio.Task in C

2016-10-28 Thread Roundup Robot
Roundup Robot added the comment: New changeset fb7c439103b9 by Victor Stinner in branch '3.6': Issue #28544: Fix _asynciomodule.c on Windows https://hg.python.org/cpython/rev/fb7c439103b9 -- ___ Python tracker

[issue28199] Compact dict resizing is doing too much work

2016-10-28 Thread STINNER Victor
STINNER Victor added the comment: Remark about perf timeout: you can use --compare-to with the patched Python binary to check if the result is significant and compute the "x.xx faster" number. -- ___ Python tracker

[issue28522] can't make IDLEX work with python._pth and python-3.6.0b2

2016-10-28 Thread Terry J. Reedy
Terry J. Reedy added the comment: Yes, see the commit to branch 3.6, which will next be released as .0b3. -- ___ Python tracker ___

[issue28522] can't make IDLEX work with python._pth and python-3.6.0b2

2016-10-28 Thread Big Stone
Big Stone added the comment: will it be in python-3.6.0b3 ? what should be in python._pth, in WinPython particular case ? (as Lib\site-packages didn't seem needed, for unknown reason) -- ___ Python tracker

[issue28549] curses: calling addch() with an 1-length str segfaults with ncurses6 compiled with --enable-ext-colors

2016-10-28 Thread Serhiy Storchaka
Serhiy Storchaka added the comment: The _curses module uses direct access to cchar_t fields. Proposed patch makes it using setcchar. Does it fixes this issue Yutao? -- keywords: +patch nosy: +serhiy.storchaka, twouters stage: -> patch review versions: +Python 3.6, Python 3.7 -Python

[issue28551] sysconfig.py wrong _PROJECT_BASE for Py2.7 Windows 64bit PC/VS9.0

2016-10-28 Thread Anselm Kruis
New submission from Anselm Kruis: Affected versions: 2.7.11, 2.7.12 Windows amd64 build with Visual Studio 2008 using the files in PC/VS9.0 If you run test_sysconfig using PC/VS9.0/rt.bat the test case test_get_config_h_filename fails.

[issue28544] Implement asyncio.Task in C

2016-10-28 Thread STINNER Victor
STINNER Victor added the comment: Python 3.6 & default don't compile on Windows because of the change: http://buildbot.python.org/all/builders/AMD64%20Windows8%203.x/builds/2795/steps/compile/logs/stdio "D:\buildarea\3.x.bolen-windows8\build\PCbuild\_asyncio.vcxproj" (Build target)

[issue28546] Better explain setting pdb breakpoints

2016-10-28 Thread Ian Kelling
New submission from Ian Kelling: https://docs.python.org/3.7/library/pdb.html: "The typical usage to break into the debugger from a running program is to insert import pdb; pdb.set_trace() at the location you want to break into the debugger." A plain read of this says: insert code into a

[issue28547] Python to use Windows Certificate Store

2016-10-28 Thread Jean-Philippe Landry
New submission from Jean-Philippe Landry: Hello, Would it be possible for Python to use the Certificate Store in windows instead of a predetermined list of certificates. The use case is as follows: Multiple machines being on a corporate network where there is a man in the middle packet

[issue28547] Python to use Windows Certificate Store

2016-10-28 Thread Christian Heimes
Christian Heimes added the comment: Python's ssl library has used Windows' cert store since 3.4 / 2.7.9. Some third party applications like requests or pip rather use their own cert store instead. This decision is beyond control of Python.

[issue26862] android: SYS_getdents64 does not need to be defined on android API 21

2016-10-28 Thread cschramm
cschramm added the comment: Any plans to fix this in 3.5 as well? -- nosy: +cschramm ___ Python tracker ___

[issue28545] socket.bind to AF_PACKET should use passed interface name

2016-10-28 Thread Martin Turon
Martin Turon added the comment: If I bind to port 0, it works for both tx and rx. Sorry for the false alarm! sock.bind(("monitor0", 0)) -- status: open -> closed ___ Python tracker

[issue28353] os.fwalk() unhandled exception when error occurs accessing symbolic link target

2016-10-28 Thread Serhiy Storchaka
Serhiy Storchaka added the comment: test_walk_bad_dir renamed one of subdirs and tearDown failed to clean up changed tree. An error happened depending on the order of listed subdirectories: ['SUB1', 'SUB2'] -- passed, ['SUB2', 'SUB1'] -- failed. --

[issue26940] android: test_importlib hangs on armv7

2016-10-28 Thread Xavier de Gaye
Xavier de Gaye added the comment: Adding a dependency to issue 26939. Patch attached. -- assignee: -> xdegaye components: +Tests -Library (Lib) dependencies: +android: test_functools hangs on armv7 keywords: +patch stage: -> patch review versions: +Python 3.7 Added file:

[issue28353] os.fwalk() unhandled exception when error occurs accessing symbolic link target

2016-10-28 Thread Serhiy Storchaka
Serhiy Storchaka added the comment: Hmm, tests were passed when I ran them before committing. Seems this is heisenbug. -- ___ Python tracker ___

[issue28353] os.fwalk() unhandled exception when error occurs accessing symbolic link target

2016-10-28 Thread Xiang Zhang
Xiang Zhang added the comment: Seems like the dir you add breaks other cases. I change addCleanUp to os.chmod and get: == FAIL: test_file_like_path (test.test_os.BytesWalkTests)

[issue28353] os.fwalk() unhandled exception when error occurs accessing symbolic link target

2016-10-28 Thread Xiang Zhang
Xiang Zhang added the comment: It works. Close. :-) -- status: open -> closed ___ Python tracker ___ ___

[issue26941] android: test_threading hangs on armv7

2016-10-28 Thread Xavier de Gaye
Xavier de Gaye added the comment: Adding a dependency to issue 26939. Patch attached. -- assignee: -> xdegaye components: +Tests -Library (Lib) dependencies: +android: test_functools hangs on armv7 keywords: +patch stage: -> patch review versions: +Python 3.7 Added file:

[issue28353] os.fwalk() unhandled exception when error occurs accessing symbolic link target

2016-10-28 Thread Roundup Robot
Roundup Robot added the comment: New changeset 655510dd46fd by Serhiy Storchaka in branch '3.5': Issue #28353: Make test_os.WalkTests.test_walk_bad_dir stable. https://hg.python.org/cpython/rev/655510dd46fd New changeset df28e536f19d by Serhiy Storchaka in branch '3.6': Issue #28353: Make

[issue26939] android: test_functools hangs on armv7

2016-10-28 Thread Xavier de Gaye
Xavier de Gaye added the comment: Patch attached. -- assignee: -> xdegaye components: +Tests -Library (Lib) dependencies: -android: add platform.android_ver() stage: -> patch review versions: +Python 3.7 Added file: http://bugs.python.org/file45249/setswitchinterval.patch

[issue28541] Improve test coverage for json library - loading bytes

2016-10-28 Thread Serhiy Storchaka
Serhiy Storchaka added the comment: In general LGTM, but I added few minor comments on Rietveld. Click on the "review" near your patch. You should also receive an email notification, but it can be moved to your Spam folder. -- ___ Python tracker

[issue28046] Remove the concept of platform-specific directories

2016-10-28 Thread Roundup Robot
Roundup Robot added the comment: New changeset 86577b7100a4 by Xavier de Gaye in branch '3.6': Issue #28046: Fix the removal of the sysconfigdata module https://hg.python.org/cpython/rev/86577b7100a4 New changeset 28205c4cf20b by Xavier de Gaye in branch 'default': Issue #28046: Merge with 3.6.

[issue28199] Compact dict resizing is doing too much work

2016-10-28 Thread Xiang Zhang
Xiang Zhang added the comment: I doubt how many memcpy could benefit. Two pass does not necessarily make faster. I make a simple test: With dictresize3, (I make insert_index inline): [bin]$ ./python3 -m perf timeit -s 'd = {i:i for i in range(6)}' 'dict(d)' Median +- std

[issue27838] test_os.test_chown() failure on koobs-freebsd-current

2016-10-28 Thread Serhiy Storchaka
Serhiy Storchaka added the comment: Kubilay, could you please run tests with my patch? -- ___ Python tracker ___

[issue28199] Compact dict resizing is doing too much work

2016-10-28 Thread Xiang Zhang
Xiang Zhang added the comment: Hmm, what's the advantage? -- ___ Python tracker ___ ___ Python-bugs-list

[issue26856] android does not have pwd.getpwall()

2016-10-28 Thread Xavier de Gaye
Changes by Xavier de Gaye : -- assignee: -> xdegaye components: +Tests stage: -> patch review versions: +Python 3.7 ___ Python tracker

[issue28199] Compact dict resizing is doing too much work

2016-10-28 Thread Serhiy Storchaka
Serhiy Storchaka added the comment: What if split copying entries from inserting indices? First fill a continuous array of entries (could use memcpy if there were not deletions), then build a hashtable of continuous indices. Following patch implements this idea. -- Added file:

[issue27838] test_os.test_chown() failure on koobs-freebsd-current

2016-10-28 Thread Kubilay Kocak
Kubilay Kocak added the comment: Serhiy, ah I thought the patch would be applied to say the 'custom' builder for this buildbot or the branch in general :) If not, I can test this in ~<= 2 days locally, though its worth noting that the issue quite likely not be reproducible outside of the

[issue28199] Compact dict resizing is doing too much work

2016-10-28 Thread INADA Naoki
INADA Naoki added the comment: LGTM. -- ___ Python tracker ___ ___ Python-bugs-list mailing list Unsubscribe:

[issue28548] http.server parse_request() bug and error reporting

2016-10-28 Thread Barry A. Warsaw
New submission from Barry A. Warsaw: This might also affect other Python version; I haven't checked, but I know it affects Python 3.5. In Mailman 3, we have a subclass of WSGIRequestHandler for our REST API, and we got a bug report about an error condition returning a 400 in the body of the

[issue28199] Compact dict resizing is doing too much work

2016-10-28 Thread Serhiy Storchaka
Serhiy Storchaka added the comment: Yes, the difference is pretty small. The largest difference I got is 7% in following microbenchmark: $ ./python -m perf timeit -s 'x = list(range(1000))' -- 'dict.fromkeys(x)' Unpatched: Median +- std dev: 80.6 us +- 0.5 us dictresize3.patch:

[issue28522] can't make IDLEX work with python._pth and python-3.6.0b2

2016-10-28 Thread Eryk Sun
Eryk Sun added the comment: > ignore PYTHONPATH and the current working directory Generally it's the script directory that isolated mode removes from sys.path. It's the working directory when there is no script. -- ___ Python tracker

[issue28544] Implement asyncio.Task in C

2016-10-28 Thread STINNER Victor
STINNER Victor added the comment: Since asyncio becomes popular, IMO it is worth to mention this optimization at: https://docs.python.org/dev/whatsnew/3.6.html#optimizations Can you please do it Yury? -- ___ Python tracker

[issue28544] Implement asyncio.Task in C

2016-10-28 Thread Yury Selivanov
Yury Selivanov added the comment: > Can you please do it Yury? Yes, Elvis and I will be the editors of What's New this year anyways ;) -- ___ Python tracker

[issue28128] Improve the warning message for invalid escape sequences

2016-10-28 Thread Nick Coghlan
Nick Coghlan added the comment: Petr, is there any chance you or someone from your team could take a look at this and the other issue Emanuel referenced? These warnings are likely to pose a usability problem when upgrading Python in Fedora in their current state, so it would be good to see

[issue28552] Distutils fail if sys.executable is None

2016-10-28 Thread Alexander P
New submission from Alexander P: For example, Jython 2.7. When we try to execute `jython-standalone -m pip`, distutils.sysconfig tries to parse sys.executable as a path and obviously fails: Traceback (most recent call last): File

[issue26578] Bad BaseHTTPRequestHandler response when using HTTP/0.9

2016-10-28 Thread Barry A. Warsaw
Changes by Barry A. Warsaw : -- nosy: +barry ___ Python tracker ___ ___ Python-bugs-list

[issue28544] Implement asyncio.Task in C

2016-10-28 Thread Roundup Robot
Roundup Robot added the comment: New changeset db7bcd92cf85 by Yury Selivanov in branch '3.6': Issue #28544: Pass `PyObject*` to _PyDict_Pop, not `PyDictObject*` https://hg.python.org/cpython/rev/db7bcd92cf85 New changeset 60b6e820abe5 by Yury Selivanov in branch 'default': Merge 3.6 (issue

[issue28548] http.server parse_request() bug and error reporting

2016-10-28 Thread Martin Panter
Martin Panter added the comment: I think you should be able to reproduce this without Mailman or tox, by just running “python -m http.server”. The problem is the “HTTP/0.9” protocol that Python is assuming does not include a header section, so there is no place to put a 400 status code or

[issue28548] http.server parse_request() bug and error reporting

2016-10-28 Thread Martin Panter
Martin Panter added the comment: Parsing the version from words[-1] rather than words[2] may be a minor improvement however. -- ___ Python tracker ___

[issue10721] Remove HTTP 0.9 server support

2016-10-28 Thread Barry A. Warsaw
Changes by Barry A. Warsaw : -- nosy: +barry ___ Python tracker ___ ___ Python-bugs-list

[issue28553] int.to_bytes docs logic error

2016-10-28 Thread Ofek Lev
New submission from Ofek Lev: https://docs.python.org/3/library/stdtypes.html#int.to_bytes To convert an int to the exact number of bytes required, the docs recommend "x.to_bytes((x.bit_length() // 8) + 1, ...)". This is incorrect when length is a multiple of 8, e.g. 296. The correct way is

[issue28544] Implement asyncio.Task in C

2016-10-28 Thread Roundup Robot
Roundup Robot added the comment: New changeset 635f5854cb3e by Yury Selivanov in branch '3.6': Issue #28544: Fix compilation of _asynciomodule.c on Windows https://hg.python.org/cpython/rev/635f5854cb3e New changeset 9d95510dc203 by Yury Selivanov in branch 'default': Merge 3.6 (issue #28544)

[issue28544] Implement asyncio.Task in C

2016-10-28 Thread Yury Selivanov
Yury Selivanov added the comment: > Python 3.6 & default don't compile on Windows because of the change: Pushed a fix for that. -- ___ Python tracker

[issue28548] http.server parse_request() bug and error reporting

2016-10-28 Thread Barry A. Warsaw
Barry A. Warsaw added the comment: On Oct 28, 2016, at 10:42 PM, Martin Panter wrote: >I think you should be able to reproduce this without Mailman or tox, by just >running “python -m http.server”. Yep, indeed. >The problem is the “HTTP/0.9” protocol that Python is assuming does not >include

[issue28548] http.server parse_request() bug and error reporting

2016-10-28 Thread Barry A. Warsaw
Barry A. Warsaw added the comment: On Oct 28, 2016, at 10:57 PM, Martin Panter wrote: >Parsing the version from words[-1] rather than words[2] may be a minor >improvement however. Indeed; I thought about that too. -- ___ Python tracker

[issue27860] Improvements to ipaddress module

2016-10-28 Thread Serhiy Storchaka
Changes by Serhiy Storchaka : -- versions: +Python 3.7 -Python 3.6 ___ Python tracker ___

[issue28444] Missing extensions modules when cross compiling python 3.5.2 for arm on Linux

2016-10-28 Thread Martin Panter
Martin Panter added the comment: Looks good to me :) -- ___ Python tracker ___ ___ Python-bugs-list mailing

[issue28128] Improve the warning message for invalid escape sequences

2016-10-28 Thread Emanuel Barry
Emanuel Barry added the comment: Following Ned's email to Python-Dev, I'm upping the priority of this issue to "deferred blocker". I haven't had a lot of time to work on this, but I had time to think about it, and I think that resolving #28028 would be the best way to solved this. I pinged

[issue28028] Convert warnings to SyntaxWarning in parser

2016-10-28 Thread Emanuel Barry
Emanuel Barry added the comment: Cross-ping from #28128 I think it would be best to solve this issue, solving both #28128 and any future warning-introducing feature or deprecation. I've marked this issue as a dependency for #28128 as well as upping the priority. There are less than 7 weeks

[issue25660] tabs don't work correctly in python repl

2016-10-28 Thread Martin Panter
Martin Panter added the comment: Clément: Yes, that sounds like the intended behaviour, at least when using Readline. Originally discussed in Issue 23441 and Issue 22086. (Though I think the change in behaviour when completion is called outside of Readline is a bug.) --

[issue28552] Distutils fail if sys.executable is None

2016-10-28 Thread R. David Murray
R. David Murray added the comment: I'm not sure it is reasonable to expect distutils to work if sys.executable is not valid. What do you think distutils could do instead? -- nosy: +r.david.murray ___ Python tracker

[issue26862] android: SYS_getdents64 does not need to be defined on android API 21

2016-10-28 Thread Chi Hsuan Yen
Chi Hsuan Yen added the comment: cschramm: AFAIK only Python 3.6+ has experimental Android support. 3.5 or below are not supported. -- nosy: +Chi Hsuan Yen ___ Python tracker

[issue27860] Improvements to ipaddress module

2016-10-28 Thread Nick Coghlan
Nick Coghlan added the comment: Peter, are you able to take a look at this or indicate you're happy for someone else to take it? (I relinquished my co-maintainer role for the ipaddress module a while back, so you're the only currently listed maintainer) --

[issue26862] android: SYS_getdents64 does not need to be defined on android API 21

2016-10-28 Thread Xavier de Gaye
Xavier de Gaye added the comment: Sorry, AFAIK there is no plan to retrofit the Android changes to 3.5. Note that Python 3.6 is expected to be released in few weeks, by next mid-december. -- ___ Python tracker

[issue10721] Remove HTTP 0.9 server support

2016-10-28 Thread Martin Panter
Martin Panter added the comment: Here is a slightly updated patch I have been sitting on. I think the main difference was tweaks to the documentation. Issue 28548 would also benefit if the HTTP 0.9 code was removed. The presence of the HTTP 0.9 code means a suboptimal response is sent in some

[issue28544] Implement asyncio.Task in C

2016-10-28 Thread Yury Selivanov
Yury Selivanov added the comment: Looks like that Windows buildbot is green now. Closing. -- resolution: -> fixed status: open -> closed ___ Python tracker

[issue28548] http.server parse_request() bug and error reporting

2016-10-28 Thread Martin Panter
Martin Panter added the comment: Here is a patch to parse the version from the request in more cases. Since it is more of a cosmetic improvement for handling erroneous requests, I would probably only apply it to the next Python version (3.7 atm). Or do you think it should go into bug fix

[issue26578] Bad BaseHTTPRequestHandler response when using HTTP/0.9

2016-10-28 Thread Martin Panter
Martin Panter added the comment: Since my last comment, I have become more confident that Python’s request parsing never supported proper HTTP 0.9. So I would prefer to remove it in the next version of Python, and not bother fixing Xiang’s bug in existing versions (unless someone offers a

[issue28549] curses: calling addch() with an 1-length str segfaults with ncurses6 compiled with --enable-ext-colors

2016-10-28 Thread Yutao Yuan
Yutao Yuan added the comment: It fails to compile for me. setcchar should take a cchar_t* and a const wchar_t* instead. -- ___ Python tracker ___

[issue28539] httplib/http.client HTTPConnection._set_hostport() regression

2016-10-28 Thread Martin Panter
Martin Panter added the comment: In general, I would be against stripping square brackets if the host and port have already been separated, e.g. in the address tuple used with socket.create_connection() etc. It feels like unnecessary double processing, which should already have been done at a

[issue28542] document cross compilation

2016-10-28 Thread Martin Panter
Martin Panter added the comment: Thanks, this looks like a decent start. I’m not that familiar with cross-compiling Python, but here are a couple questions: $(cd /path/to/source && pwd)/configure Why do you use $(pwd) here? I have mainly seen relative paths used (my own experiments, and

[issue28199] Compact dict resizing is doing too much work

2016-10-28 Thread Serhiy Storchaka
Serhiy Storchaka added the comment: The advantage is using memcpy in case of combined table without deletions. This is common case of creating dict without pre-resizing: dict(list), dict(iterator), etc. In future, when will be possible to reuse old entries array, this also could help in case

[issue28199] Compact dict resizing is doing too much work

2016-10-28 Thread INADA Naoki
INADA Naoki added the comment: Current code and my patch called insertdict_clean() or insert_index() for each entry. On the other hand, Serhiy's patch calls build_indices() once. This may be faster when compiler doesn't inlining the helper function. As a bonus, we can use memcpy to copy

[issue27860] Improvements to ipaddress module

2016-10-28 Thread Moritz Sichert
Moritz Sichert added the comment: Any updates? -- ___ Python tracker ___ ___ Python-bugs-list mailing list

[issue28541] Improve test coverage for json library - loading bytes

2016-10-28 Thread Eric Appelt
Eric Appelt added the comment: I was able to inspect the review, and implemented your suggestions. This is attached as a new patch. Please let me know if this is not the correct workflow. Thank you for the prompt review and help as I learn the python tracking and review process! --