[issue30415] Improve fnmatch testing

Changes by Serhiy Storchaka : -- pull_requests: +1787 ___ Python tracker ___ ___

[issue19495] context manager for measuring duration of blocks of code

Changes by Guido van Rossum : -- nosy: -gvanrossum ___ Python tracker ___ ___

[issue30415] Improve fnmatch testing

Serhiy Storchaka added the comment: New changeset 817554715b5578b0bac5c35e6ec9a1abd8649e53 by Serhiy Storchaka in branch 'master': bpo-30415: Add new tests for the fnmatch module. (#1684) https://github.com/python/cpython/commit/817554715b5578b0bac5c35e6ec9a1abd8649e53 --

[issue30420] Clarify kwarg handing for subprocess convenience APIs

Martin Panter added the comment: For the curious, Nick added the “frequently used arguments” in Issue 13237. Before that the signatures were like : subprocess.call(*popenargs, **kwargs) -- nosy:

[issue29976] urllib.parse clarify what ' ' in schemes mean

Changes by Senthil Kumaran : -- resolution: -> fixed stage: needs patch -> resolved status: open -> closed ___ Python tracker ___

[issue29976] urllib.parse clarify what ' ' in schemes mean

Changes by Senthil Kumaran : -- pull_requests: +1786 ___ Python tracker ___ ___

[issue29976] urllib.parse clarify what ' ' in schemes mean

Changes by Senthil Kumaran : -- pull_requests: +1785 ___ Python tracker ___ ___

[issue30359] Annotating a function as returning an (async) context manager?

Changes by Jelle Zijlstra : -- nosy: +Jelle Zijlstra ___ Python tracker ___ ___

[issue29710] Incorrect representation caveat on bitwise operation docs

Changes by Sanyam Khurana : -- pull_requests: +1784 ___ Python tracker ___ ___

[issue30418] test_communicate_epipe() of test_subprocess fails randomly on AMD64 Windows7 SP1 3.x

Eryk Sun added the comment: Addressing the underlying problem would require rewriting _Py_read and _Py_write_impl to directly call ReadFile and WriteFile instead of the CRT read and write functions. Barring that, on Windows _stdin_write would have to always ignore EINVAL. --

[issue25720] Fix curses module compilation with ncurses6

Masayuki Yamamoto added the comment: Hi, I finished various things and tackle the issue again, I opened PR 1689 at last. Changes from previous patch: * If ncurses doesn't have both is_pad function and _flags field of WINDOW, NCURSES_OPAQUE is defined as zero to make WINDOW to non-opaque type

[issue12978] Figure out extended attributes on BSDs

Changes by Roundup Robot : -- pull_requests: +1783 ___ Python tracker ___

[issue25720] Fix curses module compilation with ncurses6

Changes by Masayuki Yamamoto : -- pull_requests: +1782 ___ Python tracker ___ ___

[issue30420] Clarify kwarg handing for subprocess convenience APIs

Changes by Alex Gaynor : -- pull_requests: +1781 ___ Python tracker ___ ___

[issue30420] Clarify kwarg handing for subprocess convenience APIs

New submission from Nick Coghlan: As per the discussion in https://github.com/python/cpython/pull/1685, the current nominal signatures of the convenience APIs in subprocess is confusing, as they don't make it clear that all Popen keyword arguments are supported, with only the most common ones

[issue30418] test_communicate_epipe() of test_subprocess fails randomly on AMD64 Windows7 SP1 3.x

STINNER Victor added the comment: Would you like to work on a PR Eryk? It seems like you understand the bug better than me ;-) -- ___ Python tracker ___

[issue30409] locale.getpreferredencoding doesn't return result

STINNER Victor added the comment: Sean McCully: Thank you very much for both your bug report and for your pull request! You got your name into Python ;-) -- resolution: -> fixed status: open -> closed ___ Python tracker

[issue30418] test_communicate_epipe() of test_subprocess fails randomly on AMD64 Windows7 SP1 3.x

Eryk Sun added the comment: > but the error is only ignored when the child process is still alive. This should instead be the error is only ignored when child process is *no longer* alive. -- ___ Python tracker

[issue30418] test_communicate_epipe() of test_subprocess fails randomly on AMD64 Windows7 SP1 3.x

Eryk Sun added the comment: Seeing EINVAL here while the child process is alive could mean the read end of the pipe was closed. For example: >>> import time, subprocess >>> cmd = 'python -c "import os, time; os.close(0); time.sleep(15)"' >>> p = subprocess.Popen(cmd,

[issue30409] locale.getpreferredencoding doesn't return result

STINNER Victor added the comment: New changeset cef8b1741c03a09ea7371947dc65f632c61ef8ad by Victor Stinner (Sean McCully) in branch '2.7': bpo-30409: locale.getpreferredencoding doesn't return result (#1672) https://github.com/python/cpython/commit/cef8b1741c03a09ea7371947dc65f632c61ef8ad

[issue30290] IDLE: add tests for help_about.py

Cheryl Sabella added the comment: Terry, Is there interest in changing the 'from tkinter import *'? -- ___ Python tracker ___

[issue30410] Documentation for sys.stdout encoding does not reflect the new Windows behavior in Python 3.6+

Eryk Sun added the comment: I discussed character devices mostly because of the NUL device. It could be surprising that Python dies on an encoding error when output is redirected to NUL: C:\>chcp 1252 Active code page: 1252 C:\>python -c "print('\u20ac')" > nul C:\>chcp 437

[issue30302] Improve .__repr__ implementation for datetime.timedelta

Martin Panter added the comment: I think the benefit of the repr being easier to understand outweighs the pain of breaking the old format. If the change is a problem, that might be mitigated by adding an entry to the “Porting to Python 3.7” documentation. I don’t think my option of factoring

[issue30410] Documentation for sys.stdout encoding does not reflect the new Windows behavior in Python 3.6+

Steve Dower added the comment: Looks great, though I wonder whether the rest of the paragraph after "Character devices such as NUL" would be more confusing than it's worth? Can you create a PR? (And having links to the environment variable docs would be great.) --

[issue30362] Launcher add list and list with paths options

Steve Dower added the comment: I agree with printing discovered interpreters in help text - not a fan of the -0 option. IDEs should probably go directly to the registry to discover Python installations (or use pep514tools), since the launcher will only launch a subset of them (i.e. per-user

[issue30405] build.bat: register binaries for py launcher

Steve Dower added the comment: I am most interested in making py.exe follow PEP 514 and look up tags. Given this, we could register builds as "PythonCore\Dev" and then use "py.exe -dev" to run the most recent build. At worst, we only have one lingering registration (and a positive - it would

[issue30290] IDLE: add tests for help_about.py

Cheryl Sabella added the comment: This looks like fun! :-) I'll let you know if I have any questions. -- ___ Python tracker ___

[issue30409] locale.getpreferredencoding doesn't return result

Changes by Cheryl Sabella : -- pull_requests: +1780 ___ Python tracker ___ ___

[issue30419] Bdb: Update doc page

New submission from Cheryl Sabella: Update bdb docs to reflect issues found in #30211. -- assignee: docs@python components: Documentation messages: 294057 nosy: csabella, docs@python, terry.reedy priority: normal severity: normal status: open title: Bdb: Update doc page type:

[issue30418] test_communicate_epipe() of test_subprocess fails randomly on AMD64 Windows7 SP1 3.x

Changes by STINNER Victor : -- components: +Tests ___ Python tracker ___ ___

[issue30418] test_communicate_epipe() of test_subprocess fails randomly on AMD64 Windows7 SP1 3.x

New submission from STINNER Victor: Hum, the following failing looks like bpo-19612, but it seems like the code took the "raise" path instead of the "pass" fails. try: self.stdin.write(input) < HERE except BrokenPipeError: pass #

[issue30351] regrtest hangs on x86 Windows XP 2.7

STINNER Victor added the comment: Hum, it's the second time that regrtest hangs on x86 Windows XP 2.7 while test_threading is running. -- ___ Python tracker

[issue30351] regrtest hangs on x86 Windows XP 2.7

STINNER Victor added the comment: New fail on x86 Windows XP 2.7: http://buildbot.python.org/all/builders/x86%20Windows%20XP%202.7/builds/166/steps/test/logs/stdio ... 0:35:19 [401/403] test_email_codecs passed -- running: test_threading (70 sec) 0:35:20 [402/403] test_gettext passed --

[issue30417] [buildbot] Disable the cpu resources on slow buildbots

Changes by Zachary Ware : -- nosy: +zach.ware ___ Python tracker ___ ___

[issue30314] Buildbots: 15 min is too low for test_tools on x86 Tiger 3.6 buildbot

Changes by Zachary Ware : -- nosy: +zach.ware ___ Python tracker ___ ___

[issue30314] Buildbots: 15 min is too low for test_tools on x86 Tiger 3.6 buildbot

STINNER Victor added the comment: I merged my PR and applied the new buildbot configuration. I will try to check if the config was applied correctly. Serhiy: "test_tools is so slow because it proceeds *all* Python files. This controlled by the "cpu" resource (only 10 random files are chosen

[issue30417] [buildbot] Disable the cpu resources on slow buildbots

New submission from STINNER Victor: Follow-up of the issue #30314. Copy of Serhiy Storchaka's msg294048: -- test_tools is so slow because it proceeds *all* Python files. This controlled by the "cpu" resource (only 10 random files are chosen if it is disabled). Maybe disable the "cpu" resource

[issue30416] constant folding opens compiler to quadratic time hashing

Serhiy Storchaka added the comment: Nice example. The only fix I see is caching the hash in a tuple. This can even help in more cases, when tuples are used as dict keys. But this affect too much code, and can even break a code that mutates a tuple with refcount 1. -- nosy:

[issue30416] constant folding opens compiler to quadratic time hashing

New submission from Andrew Dalke: Others have reported issues like #21074 where the peephole compiler generates and discards large strings, and #30293 where it generates multi-MB integers and stores them in the .pyc. This is a different issue. The code: def tuple20(): return

[issue23560] Group the docs of similar methods in stdtypes.rst

Cheryl Sabella added the comment: Hi Martin, Do you think it would be good to move forward with this as a PR? Thanks! -- nosy: +csabella ___ Python tracker

[issue30314] Buildbots: 15 min is too low for test_tools on x86 Tiger 3.6 buildbot

Serhiy Storchaka added the comment: test_tools is so slow because it proceeds *all* Python files. This controlled by the "cpu" resource (only 10 random files are chosen if it is disabled). Maybe disable the "cpu" resource on slow buildbots? -- nosy: +serhiy.storchaka

[issue30314] Buildbots: 15 min is too low for test_tools on x86 Tiger 3.6 buildbot

STINNER Victor added the comment: PR: https://github.com/python/buildmaster-config/pull/8 -- ___ Python tracker ___

[issue30415] Improve fnmatch testing

Changes by Serhiy Storchaka : -- pull_requests: +1779 ___ Python tracker ___ ___

[issue30413] Add fnmatch.filter_false function

Changes by Serhiy Storchaka : -- components: +Library (Lib) dependencies: +Improve fnmatch testing ___ Python tracker ___

[issue30415] Improve fnmatch testing

New submission from Serhiy Storchaka: Proposed patch adds new tests for the fnmatch module. -- assignee: serhiy.storchaka components: Tests messages: 294045 nosy: serhiy.storchaka priority: normal severity: normal stage: patch review status: open title: Improve fnmatch testing type:

[issue30410] Documentation for sys.stdout encoding does not reflect the new Windows behavior in Python 3.6+

Eryk Sun added the comment: How about this? The character encoding is platform-dependent. Non-Windows platforms use the locale encoding (see locale.getpreferredencoding()). On Windows, UTF-8 is used for console character devices (i.e. CON, CONIN$, and CONOUT$). However,

[issue30414] multiprocesing.Queue silently ignore messages after exc in _feeder

Changes by Grzegorz Grzywacz : -- pull_requests: +1778 ___ Python tracker ___ ___

[issue30414] multiprocesing.Queue silently ignore messages after exc in _feeder

New submission from Grzegorz Grzywacz: multiprocessing.Queue is running background thread feeder. Feeder serialize and sends buffered data to pipe. The issue is with exception handling, feeder is catching all exceptions but out of main loop, so after exception is handled feeder is not going

[issue23667] IDLE to provide option for making trailing whitespace visible

Cheryl Sabella added the comment: Thanks, Terry! I'll take a look at 30290. No need to start with colorizer; it was in this ticket so I referenced it more for convenience than anything. -- ___ Python tracker

[issue23674] super() documentation isn't very clear

Cheryl Sabella added the comment: Thank you for pointing out those other links. For me, they all tie together because I was originally trying to figure out why the data model page defined `super(B, obj).m()` instead of just `super().m()`. It seems that the zero argument super is preferred,

[issue30413] Add fnmatch.filter_false function

Serhiy Storchaka added the comment: Yes, it does so only if normalize_case is True. Currently fnmatch.filter() returns non-normalized names. -- ___ Python tracker

[issue30388] ndbm can't iterate through values on OS X

Ned Deily added the comment: Can you provide a complete example of how to reproduce the behavior you are seeing? Ideally, it would include producing the shelve object. Also, please provide the output of this command: otool -L $(python3.6 -c 'import _dbm;print(_dbm.__file__)') --

[issue30413] Add fnmatch.filter_false function

Wolfgang Maier added the comment: Does it? I thought it does so only if normalize_case is True. Did I miss something? -- ___ Python tracker ___

[issue30413] Add fnmatch.filter_false function

Serhiy Storchaka added the comment: Your patch makes filter() returning normalized names. This may be not what is expected. -- ___ Python tracker ___

[issue27945] Various segfaults with dict

Serhiy Storchaka added the comment: Thank you for your patches Duane and Tim. Thank you for your very detailed report that allow writing these patches tehybel. I excluded changes in dict.fromkeys() since they look unnecessary for this issue after fixing insertdict(). There are other reasons

[issue30413] Add fnmatch.filter_false function

Wolfgang Maier added the comment: Hi, seems I had the same thoughts as you, Steven. I had started working on a patch independently yesterday, but after making my changes to fnmatch itself, I found I had too many other things to do to address unittests and doc changes to turn this into a real

[issue27945] Various segfaults with dict

Serhiy Storchaka added the comment: New changeset e6a0b5982973e64b9fa28e5e3e54eb8c47882780 by Serhiy Storchaka in branch '2.7': [2.7] bpo-27945: Fixed various segfaults with dict. (GH-1657) (#1681) https://github.com/python/cpython/commit/e6a0b5982973e64b9fa28e5e3e54eb8c47882780 --

[issue30245] possible overflow when organize struct.pack_into error message

Changes by Serhiy Storchaka : -- components: +Extension Modules stage: needs patch -> patch review ___ Python tracker ___

[issue30245] possible overflow when organize struct.pack_into error message

Changes by Yuan Liu : -- pull_requests: +1777 ___ Python tracker ___ ___

[issue29619] st_ino (unsigned long long) is casted to long long in posixmodule.c:_pystat_fromstructstat

STINNER Victor added the comment: I'm fine with always using unsigned long long. I don't think that it's even possible to measure the perf difference and if it's doable, I expect don't expect it to be significant... especially compared to the cost of the stat () syscall! Well, most computers

[issue27945] Various segfaults with dict

Changes by Serhiy Storchaka : -- pull_requests: +1776 ___ Python tracker ___ ___

[issue30362] Launcher add list and list with paths options

Eryk Sun added the comment: I'm -0 on the "-0" option, but I think printing the list of registered Python installations would be useful in the --help text and when the launcher fails to find a requested version. -- nosy: +eryksun ___ Python tracker

[issue30403] Running extension modules using -m switch

Nick Coghlan added the comment: Yep, that's the kind of thing we'd like to make possible. -- ___ Python tracker ___

[issue30413] Add fnmatch.filter_false function

Serhiy Storchaka added the comment: In general LGTM. I left few minor comments on Rietveld. Needed the versionadded directive, Misc/NEWS and What's New entries. But shouldn't the new function have name "filterfalse" for parallel with itertools.filterfalse? And this would better match the

[issue29619] st_ino (unsigned long long) is casted to long long in posixmodule.c:_pystat_fromstructstat

Xavier de Gaye added the comment: Good point. PyLong_FromLong() is faster than PyLong_FromLongLong(), IMO the fastest should be used when it is possible. PyLong_FromUnsignedLong() and PyLong_FromUnsignedLongLong() are almost identical, they implement the same algorithm. The first one uses

[issue20751] Misleading descriptor protocol documentation: direct call, super binding

Changes by Mark Lawrence : -- nosy: -BreamoreBoy ___ Python tracker ___ ___

[issue30413] Add fnmatch.filter_false function

New submission from Steven D'Aprano: There has been some discussion on Python-Ideas about adding fnmatch.filter_false to complement filter, for when you want to ignore matching files rather than non-matching ones. https://mail.python.org/pipermail/python-ideas/2017-May/045694.html I don't

[issue9584] fnmatch, glob: Allow curly brace expansion

Changes by Steven D'Aprano : -- nosy: +steven.daprano ___ Python tracker ___ ___

[issue30412] mailbox : add option to prevent platform EOL conversion

New submission from adder: The mailbox module will actively convert EOL format from the input mailbox, towards the EOL convention of the platform it is running on. This is wrong. If the mailbox originates from the same platform, as the code is executing on, it is an unnecessary conversion.

[issue19495] context manager for measuring duration of blocks of code

Changes by irdb : -- nosy: +irdb ___ Python tracker ___ ___ Python-bugs-list mailing

[issue8704] cgitb sends a bogus HTTP header if the app crashes before finishing headers

Артур Клесун added the comment: Apache started strict check of headers ch aracters to be valid recently. That causes it fail on "<--: spam". ``` [Sat May 20 13:09:23.056673 2017] [http:error] [pid 26379] [client 12.34.567.41:60988] AH02429: Response header name

[issue23674] super() documentation isn't very clear

Martin Panter added the comment: Cheryl: see also Issue 25777 and Issue 20751, both about the “super binding” under Invoking Descriptors. Raymond: if you want to just pick parts of my patch, go for it. But I don’t understand your concern about explaining the MRO. I think it is important to

[issue30411] git doesn't support "-C" args under 1.8.5 occurs in configure.ac

Changes by Roundup Robot : -- pull_requests: +1775 ___ Python tracker ___

[issue27945] Various segfaults with dict

Serhiy Storchaka added the comment: New changeset 564398af6ccb34d0db8b6e2537830eca285689e5 by Serhiy Storchaka in branch '3.6': [3.6] bpo-27945: Fixed various segfaults with dict. (GH-1657) (#1677) https://github.com/python/cpython/commit/564398af6ccb34d0db8b6e2537830eca285689e5 --

[issue27945] Various segfaults with dict

Serhiy Storchaka added the comment: New changeset 2f7f533cf6fb57fcedcbc7bd454ac59fbaf2c655 by Serhiy Storchaka in branch '3.5': [3.5] bpo-27945: Fixed various segfaults with dict. (GH-1657) (#1678) https://github.com/python/cpython/commit/2f7f533cf6fb57fcedcbc7bd454ac59fbaf2c655 --

[issue30411] git doesn't support "-C" args under 1.8.5 occurs in configure.ac

Changes by Roundup Robot : -- pull_requests: +1774 ___ Python tracker ___

[issue27945] Various segfaults with dict

Changes by Serhiy Storchaka : -- pull_requests: +1773 ___ Python tracker ___ ___

[issue27945] Various segfaults with dict

Changes by Serhiy Storchaka : -- pull_requests: +1772 ___ Python tracker ___ ___

[issue20751] Misleading descriptor protocol documentation: direct call, super binding

Martin Panter added the comment: Lower-case “a” is defined at the top of the list: “The starting point . . . is ‘a.x’.” The last entry may fit in better if it was written “If binding to an instance of ‘super’ ”. The problem with the m() call is also mentioned in Issue 25777, about the

[issue27945] Various segfaults with dict

Serhiy Storchaka added the comment: New changeset 753bca3934a7618a4fa96e107ad1c5c18633a683 by Serhiy Storchaka in branch 'master': bpo-27945: Fixed various segfaults with dict. (#1657) https://github.com/python/cpython/commit/753bca3934a7618a4fa96e107ad1c5c18633a683 --

[issue30411] git doesn't support "-C" args under 1.8.5 occurs in configure.ac

New submission from Yuan Liu: git doesn't support "-C" args under 1.8.5 occurs in configure.ac as I noticed from git release notes in 1.8.5: * Just like "make -C ", "git -C ..." tells Git to go there before doing anything else. So we should have some conditional statements in

[issue30410] Documentation for sys.stdout encoding does not reflect the new Windows behavior in Python 3.6+

New submission from Paul Moore: The documentation for the encoding of sys.stdin/out/err (see https://docs.python.org/3.6/library/sys.html#sys.stdout) does not reflect the change in Python 3.6 on Windows to use the console Unicode APIs, and hence UTF-8 for the encoding. -- assignee:

[issue30403] Running extension modules using -m switch

Terry J. Reedy added the comment: So a use case might be that someone could compile all the stdlib .py modules with cython, as they are, without touching the code, and have the result be a drop-in replacement. I'd like that to be possible. -- ___

[issue30378] SysLogHandler does not support IPv6 destinations

Changes by Xiang Zhang : -- nosy: +vinay.sajip, xiang.zhang ___ Python tracker ___ ___

[issue30378] SysLogHandler does not support IPv6 destinations

Changes by Xiang Zhang : -- pull_requests: +1771 ___ Python tracker ___ ___

[issue25794] __setattr__ does not always overload operators

Changes by Serhiy Storchaka : -- resolution: -> fixed stage: patch review -> resolved status: open -> closed ___ Python tracker

[issue25794] __setattr__ does not always overload operators

Serhiy Storchaka added the comment: New changeset 4a86fe9d3f5e7af6f019ae22536eec228f04e22e by Serhiy Storchaka in branch '3.5': [3.5] bpo-25794: Fix `type.__setattr__()` for non-interned attribute names. (GH-1652) (#1674)

[issue25794] __setattr__ does not always overload operators

Serhiy Storchaka added the comment: New changeset 193f7e094f070cecbc6faea6dffafb80ea9e7536 by Serhiy Storchaka in branch '3.6': [3.6] bpo-25794: Fix `type.__setattr__()` for non-interned attribute names. (GH-1652) (#1673)

[issue30403] Running extension modules using -m switch

Nick Coghlan added the comment: As a high level overview of the general idea: we'd like it to be almost entirely transparent to the end user as to whether a particular module is implemented as normal Python source code, a precompiled bytecode/wordcode file, or a precompiled Cython extension

[issue25794] __setattr__ does not always overload operators

Serhiy Storchaka added the comment: New changeset e9f9b042781e3cb480315860bcdf7b7d22cba0f8 by Serhiy Storchaka in branch '2.7': [2.7] bpo-25794: Fix `type.__setattr__()` for non-interned or unicode attribute names. (GH-1652) (#1675)

[issue30362] Launcher add list and list with paths options

Steve Barnes added the comment: @terry.reedy - Very good points and I like the shortness of -0 note that there is already an accepted change to allow -x.y-64 to specifically require the 64 bit versions so we don't need the complex logic to distinguish cases where -x.y will run 32 bit because

[issue17188] Document 'from None' in raise statement doc.

Terry J. Reedy added the comment: The double listing seems to be a glitch. Either someone else does backport or online doc change waits until I can do it. -- assignee: docs@python -> terry.reedy stage: patch review -> backport needed versions: +Python 3.6, Python 3.7 -Python 3.4,

[issue23674] super() documentation isn't very clear

Raymond Hettinger added the comment: Martin, can we just fix the signature here. It is not the purpose of the super docs explain the MRO (that is a feature of all new-style classes, super() is just something that hops to the next in the chain) or to be a tutorial. I would like to keep the

[issue17188] Document 'from None' in raise statement doc.

Terry J. Reedy added the comment: New changeset 763557eac06ba60d7c5133e4f80df8870d8f917e by terryjreedy (csabella) in branch 'master': bpo-17188: DOC: Document 'from None' in raise statement (#1671) https://github.com/python/cpython/commit/763557eac06ba60d7c5133e4f80df8870d8f917e --

[issue25794] __setattr__ does not always overload operators

Changes by Serhiy Storchaka : -- pull_requests: +1770 ___ Python tracker ___ ___

[issue30409] locale.getpreferredencoding doesn't return result

Changes by Decorater : Added file: http://bugs.python.org/file46878/suggested_change.diff ___ Python tracker ___

[issue30408] [defaultdict] default_factory should accept a "key" default parameter (which can be passed my __missing__)

Nam added the comment: Sure, thanks Raymond and Steven for your thoughts. -- stage: -> resolved status: open -> closed ___ Python tracker ___

[issue30409] locale.getpreferredencoding doesn't return result

STINNER Victor added the comment: > ok, is this a valid fix then? Yes, totally. Sorry, ignore my first comment. -- ___ Python tracker ___

[issue30409] locale.getpreferredencoding doesn't return result

Sean McCully added the comment: ok,  is this a valid fix then? On Saturday, May 20, 2017 1:34 AM, STINNER Victor wrote: STINNER Victor added the comment: > In fact, it seems like I introduced a regression in bpo-6393, commit >