[issue34603] ctypes on Windows: error calling C function that returns a struct containing 3 bools

2018-09-15 Thread miss-islington
miss-islington added the comment: New changeset e53632019816749ffd5be0afab2a99d744dbbe35 by Miss Islington (bot) in branch '3.6': bpo-34603, ctypes/libffi_msvc: Fix returning structs from functions (GH-9258) https://github.com/python/cpython/commit/e53632019816749ffd5be0afab2a99d744dbbe35

[issue34603] ctypes on Windows: error calling C function that returns a struct containing 3 bools

2018-09-15 Thread miss-islington
miss-islington added the comment: New changeset e3f6aa7fe48b91f4ff619b2a51d473249d620bcb by Miss Islington (bot) in branch '3.7': bpo-34603, ctypes/libffi_msvc: Fix returning structs from functions (GH-9258) https://github.com/python/cpython/commit/e3f6aa7fe48b91f4ff619b2a51d473249d620bcb

[issue34603] ctypes on Windows: error calling C function that returns a struct containing 3 bools

2018-09-15 Thread miss-islington
Change by miss-islington : -- pull_requests: +8766 ___ Python tracker ___ ___ Python-bugs-list mailing list Unsubscribe:

[issue34603] ctypes on Windows: error calling C function that returns a struct containing 3 bools

2018-09-15 Thread STINNER Victor
STINNER Victor added the comment: New changeset 7843caeb909bd907e903606414e238db4082315a by Victor Stinner (Vladimir Matveev) in branch 'master': bpo-34603, ctypes/libffi_msvc: Fix returning structs from functions (GH-9258)

[issue34603] ctypes on Windows: error calling C function that returns a struct containing 3 bools

2018-09-15 Thread miss-islington
Change by miss-islington : -- pull_requests: +8765 ___ Python tracker ___ ___ Python-bugs-list mailing list Unsubscribe:

[issue34603] ctypes on Windows: error calling C function that returns a struct containing 3 bools

2018-09-15 Thread miss-islington
Change by miss-islington : -- pull_requests: +8764 ___ Python tracker ___ ___ Python-bugs-list mailing list Unsubscribe:

[issue34701] Asyncio documentation for recursive coroutines is lacking

2018-09-15 Thread Azaria Zornberg
Azaria Zornberg added the comment: Ah, thanks for the clarification! I first encountered this when having some issues with converting large objects to json. json.dumps happens synchronously, and when executed on an object that was dozens of MB in size, it held up everything for a fair bit

[issue34701] Asyncio documentation for recursive coroutines is lacking

2018-09-15 Thread Yury Selivanov
Yury Selivanov added the comment: The issue here is not the recursion, but rather about the fact that coroutines should actually await on IO or other activity in order for the event loop to run them cooperatively. E.g. async def foo(): await foo() doesn't really do anything

[issue34701] Asyncio documentation for recursive coroutines is lacking

2018-09-15 Thread Azaria Zornberg
Change by Azaria Zornberg : -- keywords: +patch pull_requests: +8762 stage: -> patch review ___ Python tracker ___ ___

[issue34701] Asyncio documentation for recursive coroutines is lacking

2018-09-15 Thread Azaria Zornberg
New submission from Azaria Zornberg : When an asynchronous coroutine in asyncio awaits or yields from itself, any call to the function is executed somewhat synchronously. Once the recursive coroutine begins, if it never awaits any other coroutines besides itself, nothing else will be

[issue32153] mock.create_autospec fails if an attribute is a partial function

2018-09-15 Thread Berker Peksag
Berker Peksag added the comment: Or we can special case functools.partial() in _copy_func_details() and use partial_object.func.__name__. (I'm not sure which solution is better, but we need to the same thing for __doc__ as well.) -- ___ Python

[issue32153] mock.create_autospec fails if an attribute is a partial function

2018-09-15 Thread Berker Peksag
Berker Peksag added the comment: I think option 2 is what functools.update_wrapper() does and it may be better to make create_autospec() consistent with it. Perhaps we can replace _copy_func_details() with functools.update_wrapper(): assigments = ( '__name__', '__doc__',

[issue32153] mock.create_autospec fails if an attribute is a partial function

2018-09-15 Thread Anthony Flury
Anthony Flury added the comment: It seems to me that we have three alternatives : 1. Refuse to create the mock object with a suitable Exception (rather than a crash 2. Copy the object and simply ignore the missing dunder_name (so that funcopy dunder_name is not set 3. Set funcopy

[issue34561] Replace list sorting merge_collapse()?

2018-09-15 Thread Tim Peters
Tim Peters added the comment: New version of runstack.py. - Reworked code to reflect that Python's sort uses (start_offset, run_length) pairs to record runs. - Two unbounded-integer power implementations, one using a loop and the other division. The loop version implies that, in Python's

[issue34683] Caret positioned wrong for SyntaxError reported by ast.c

2018-09-15 Thread Ammar Askar
Ammar Askar added the comment: Added a PR that is an implementation of Benjamin's suggestion. The column offset returned has been made 0-indexed and errors now point to the start of the parsed token. This makes errors like `def class` show up as def class ^ instead of def class

[issue34683] Caret positioned wrong for SyntaxError reported by ast.c

2018-09-15 Thread Ammar Askar
Change by Ammar Askar : -- keywords: +patch pull_requests: +8761 stage: -> patch review ___ Python tracker ___ ___

[issue31779] assertion failures and a crash when using an uninitialized struct.Struct object

2018-09-15 Thread Alexey Izbyshev
Alexey Izbyshev added the comment: What are the drawbacks of moving all code from __init__ to __new__, at least in master branch (3.8)? IMO it's much more robust than playing whack-a-mole both with ref/memleaks caused by repeated calls to __init__ and absence of initialization checks in all

[issue32117] Tuple unpacking in return and yield statements

2018-09-15 Thread Jordan Chapman
Jordan Chapman added the comment: Sorry, I could have sworn that I pasted my link... https://github.com/jChapman -- ___ Python tracker ___

[issue32117] Tuple unpacking in return and yield statements

2018-09-15 Thread Jordan Chapman
Jordan Chapman added the comment: Here's my GitHub account: I'll make the changes and rebase as soon as I get home. -- ___ Python tracker ___

[issue32117] Tuple unpacking in return and yield statements

2018-09-15 Thread Guido van Rossum
Guido van Rossum added the comment: Jordan, what's your GitHub account name? I hope you can check this out and make the changes I'm requesting on GitHub. -- ___ Python tracker

[issue34699] allow path-like objects in program arguments in Windows

2018-09-15 Thread Serhiy Storchaka
Serhiy Storchaka added the comment: This looks like a duplicate of issue31961. -- nosy: +serhiy.storchaka resolution: -> duplicate superseder: -> subprocess._execute_child doesn't accept a single PathLike argument for args ___ Python tracker

[issue33617] subprocess.Popen etc do not accept os.PathLike in passed sequence on Windows

2018-09-15 Thread Serhiy Storchaka
Serhiy Storchaka added the comment: This looks like a duplicate of issue31961. -- nosy: +serhiy.storchaka resolution: -> duplicate superseder: -> subprocess._execute_child doesn't accept a single PathLike argument for args ___ Python tracker

[issue34660] Replace ableist terms and pejoratives in source code.

2018-09-15 Thread Terry J. Reedy
Terry J. Reedy added the comment: I am re-opening with the scope limited to source code that will not be part of a doc review. The 2nd PR falls within this limit and I think it should be properly reviewed. I am opposed to removing 'sanity check' as it has a well-enough defined meaning

[issue34699] allow path-like objects in program arguments in Windows

2018-09-15 Thread Guo Ci Teo
Change by Guo Ci Teo : -- title: allows path-like objects in program arguments in Windows -> allow path-like objects in program arguments in Windows ___ Python tracker ___

[issue34700] typing.get_type_hints doesn't know about typeshed

2018-09-15 Thread Ben Darnell
New submission from Ben Darnell : Currently, most type annotations for the standard library are provided in typeshed rather than in the library itself. Not all consumers of type annotations are aware of typeshed, and this can cause problems. Specifically, Sphinx 1.8 accesses type hints via

[issue34685] scheduler tests for posix_spawn fail on AMD64 FreeBSD 10.x Shared 3.x

2018-09-15 Thread Pablo Galindo Salgado
Change by Pablo Galindo Salgado : -- stage: patch review -> resolved status: open -> closed ___ Python tracker ___ ___

[issue34699] allows path-like objects in program arguments in Windows

2018-09-15 Thread Guo Ci Teo
Change by Guo Ci Teo : -- keywords: +patch pull_requests: +8760 stage: -> patch review ___ Python tracker ___ ___ Python-bugs-list

[issue34699] allows path-like objects in program arguments in Windows

2018-09-15 Thread Guo Ci Teo
New submission from Guo Ci Teo : Currently, the `subprocess.Popen` function allows for path-like objects in the argument list for POSIX but not in Windows. This PR makes Windows' `subprocess.Popen` accept path-like objects in the argument list. -- components: Library (Lib) messages:

[issue34660] Remove ableist terms and pejoratives from source code and docs

2018-09-15 Thread Roundup Robot
Change by Roundup Robot : -- pull_requests: +8759 ___ Python tracker ___ ___ Python-bugs-list mailing list Unsubscribe:

[issue33608] [subinterpreters] Add a cross-interpreter-safe mechanism to indicate that an object may be destroyed.

2018-09-15 Thread Eric Snow
Change by Eric Snow : -- keywords: +patch pull_requests: +8758 stage: needs patch -> patch review ___ Python tracker ___ ___

[issue29883] asyncio: Windows Proactor Event Loop UDP Support

2018-09-15 Thread Adam Meily
Adam Meily added the comment: I've rebased onto upstream master and I fixed the CI build. -- ___ Python tracker ___ ___

[issue34515] lib2to3: support non-ASCII identifiers

2018-09-15 Thread miss-islington
miss-islington added the comment: New changeset 51dbae867e82014f9af89662977e4981463c51e8 by Miss Islington (bot) in branch '3.7': closes bpo-34515: Support non-ASCII identifiers in lib2to3. (GH-8950) https://github.com/python/cpython/commit/51dbae867e82014f9af89662977e4981463c51e8

[issue33338] [lib2to3] Synchronize token.py and tokenize.py with the standard library

2018-09-15 Thread monson
Change by monson : -- pull_requests: +8757 ___ Python tracker ___ ___ Python-bugs-list mailing list Unsubscribe:

[issue34515] lib2to3: support non-ASCII identifiers

2018-09-15 Thread Benjamin Peterson
Benjamin Peterson added the comment: New changeset 10a428b64b3f224e2ccd40ff2afb141b9b3425b1 by Benjamin Peterson (Monson Shao) in branch 'master': closes bpo-34515: Support non-ASCII identifiers in lib2to3. (GH-8950)

[issue34515] lib2to3: support non-ASCII identifiers

2018-09-15 Thread miss-islington
Change by miss-islington : -- pull_requests: +8756 ___ Python tracker ___ ___ Python-bugs-list mailing list Unsubscribe:

[issue34686] Add `-r`, as opposed to `-R` to Python core interpreter

2018-09-15 Thread Benjamin Peterson
Benjamin Peterson added the comment: On Fri, Sep 14, 2018, at 23:59, Erwan Le Pape wrote: > > Erwan Le Pape added the comment: > > Great! My only concern with that is marshalling of untrusted data at > runtime (I know, you shouldn't do that) can become a much more expensive > operation. >

[issue34698] urllib.request.Request.set_proxy doesn't (necessarily) replace type

2018-09-15 Thread Tim Burke
New submission from Tim Burke : Not sure if this is a documentation or behavior bug, but... the docs for urllib.request.Request.set_proxy (https://docs.python.org/3/library/urllib.request.html#urllib.request.Request.set_proxy) say > Prepare the request by connecting to a proxy server. *The

[issue34697] ctypes: Crash if manually-created CField instance is used

2018-09-15 Thread Alexey Izbyshev
New submission from Alexey Izbyshev : It is possible to manually create an instance of private CField type which is used by ctypes to represent fields of Structure and Union types. This instance will be uninitialized because it's normally initialized when instances of Structure/Union are

[issue34693] PEPping distutils/core.py

2018-09-15 Thread Berker Peksag
Berker Peksag added the comment: Thanks for the report and for the PR, John. Karthikeyan is correct. Many of the modules in the stdlib predate PEP 8. Also, making PEP 8-only changes would make "git blame" less useful and can cause regressions (IIRC it has happened at least once, but I don't

[issue30262] Don't expose sqlite3 Cache and Statement

2018-09-15 Thread Alexey Izbyshev
Alexey Izbyshev added the comment: Some additional motivation for removing Cache: it may be used to crash the interpreter (#31734). -- nosy: +izbyshev ___ Python tracker ___

[issue26759] PyBytes_FromObject accepts arbitrary iterable

2018-09-15 Thread Zackery Spytz
Zackery Spytz added the comment: I've created #34696 for PyByteArray_FromObject(). -- nosy: +ZackerySpytz ___ Python tracker ___

[issue34696] PyByteArray_FromObject() has undocumented (and untested) behavior

2018-09-15 Thread Zackery Spytz
Change by Zackery Spytz : -- keywords: +patch pull_requests: +8755 stage: -> patch review ___ Python tracker ___ ___

[issue34696] PyByteArray_FromObject() has undocumented (and untested) behavior

2018-09-15 Thread Zackery Spytz
New submission from Zackery Spytz : The documentation states that PyByteArray_FromObject() creates a bytearray object from an object that implements the buffer protocol, but PyByteArray_FromObject() simply calls the bytearray constructor with its argument (the buffer protocol part is not

[issue34695] sqlite3: Cache.get() crashes if Cache.__init__() was not called

2018-09-15 Thread Berker Peksag
Berker Peksag added the comment: Thanks for the report and for the patch! This is a duplicate of issue 31734. The Cache class is an implementation detail and it has no use of outside of the sqlite3 implementation as you already said. For option 3, there is an open issue: #30262. IMO, the

[issue34695] sqlite3: Cache.get() crashes if Cache.__init__() was not called

2018-09-15 Thread Berker Peksag
Change by Berker Peksag : -- stage: patch review -> resolved ___ Python tracker ___ ___ Python-bugs-list mailing list Unsubscribe:

[issue34695] sqlite3: Cache.get() crashes if Cache.__init__() was not called

2018-09-15 Thread Serhiy Storchaka
Serhiy Storchaka added the comment: This is a duplicate of issue31734. -- resolution: -> duplicate stage: patch review -> superseder: -> crash or SystemError in sqlite3.Cache in case it is uninitialized or partially initialized ___ Python

[issue30576] http.server should support HTTP compression (gzip)

2018-09-15 Thread Brett Cannon
Brett Cannon added the comment: Thanks for the understanding, Pierre (and thanks for the work on Brython!). On Sat., Sep. 15, 2018, 08:25 Pierre Quentel, wrote: > > Pierre Quentel added the comment: > > Brett, > > Thanks for taking the time, you and other core devs, to review the PR and >

[issue34695] sqlite3: Cache.get() crashes if Cache.__init__() was not called

2018-09-15 Thread Alexey Izbyshev
Change by Alexey Izbyshev : -- keywords: +patch pull_requests: +8754 stage: -> patch review ___ Python tracker ___ ___

[issue34695] sqlite3: Cache.get() crashes if Cache.__init__() was not called

2018-09-15 Thread Alexey Izbyshev
New submission from Alexey Izbyshev : sqlite3.Cache allows users to create uninitialized instances because it relies on __init__() instead of __new__() for initialization, which can be skipped. Cache.get() crashes if called on an uninitialized instance: >>> from sqlite3 import Cache >>>

[issue34693] PEPping distutils/core.py

2018-09-15 Thread John Joyce
John Joyce added the comment: Oh, I see. I did not realize that tidying would not be accepted. That explains a lot. As it offended linters based on PEPs I chose that as the best match. It is pedantic, but many linting actions are not far from compiling... they are generally parsing to

[issue30140] Binary arithmetic does not always call subclasses first

2018-09-15 Thread Alexander Belopolsky
Change by Alexander Belopolsky : -- nosy: +belopolsky ___ Python tracker ___ ___ Python-bugs-list mailing list Unsubscribe:

[issue30576] http.server should support HTTP compression (gzip)

2018-09-15 Thread Pierre Quentel
Pierre Quentel added the comment: Brett, Thanks for taking the time, you and other core devs, to review the PR and to explain why you took this decision. I am disappointed by the result, but I understand the reasons, having to face the same maintenance issues, on a much smaller scale, with

[issue34694] Dismiss To Avoid Slave/Master wording cause it easier for non English spoken programmers

2018-09-15 Thread Mariatta Wijaya
Mariatta Wijaya added the comment: There will be no further discussion about this. -- nosy: +Mariatta resolution: -> wont fix stage: -> resolved status: open -> closed ___ Python tracker

[issue34693] PEPping distutils/core.py

2018-09-15 Thread Karthikeyan Singaravelan
Karthikeyan Singaravelan added the comment: Thanks for the contribution. Is there any syntax error on using the file since you have classified it as compile error or the error messages are related to linters? On a general note style related contributions are not merged since they create

[issue34694] Dismiss To Avoid Slave/Master wording cause it easier for non English spoken programmers

2018-09-15 Thread Serhiy Storchaka
Serhiy Storchaka added the comment: Do you have any concrete propositions? -- nosy: +serhiy.storchaka ___ Python tracker ___ ___

[issue34694] Dismiss To Avoid Slave/Master wording cause it easier for non English spoken programmers

2018-09-15 Thread Jose Angel Acosta
New submission from Jose Angel Acosta : A request have been srecentrly uddenly committed to avoid Slave/Master wording in python code, I think the "issue"was not enough peer-reviewed, me having slave roots from my african and jewish heritage I dont consider this matter an Issue, but the

[issue33649] asyncio docs overhaul

2018-09-15 Thread Bumsik Kim
Change by Bumsik Kim : -- pull_requests: +8753 ___ Python tracker ___ ___ Python-bugs-list mailing list Unsubscribe:

[issue34693] PEPping distutils/core.py

2018-09-15 Thread John Joyce
New submission from John Joyce : In distutils/core.py ... Line 227 is over-indented, inconsistent with the rest of the file. Line 226 is over-indented, inconsistent with the rest of the file. Line 114 is over-indented by 2 characters, inconsistent with the rest of the file. else:

[issue34484] Unicode HOWTO incorrectly refers to Private Use Area for surrogateescape

2018-09-15 Thread A.M. Kuchling
A.M. Kuchling added the comment: Corrected in the unicode-howto-update branch being developed for issue #20906. -- assignee: docs@python -> akuchling nosy: +akuchling ___ Python tracker

[issue20906] Issues in Unicode HOWTO

2018-09-15 Thread A.M. Kuchling
Change by A.M. Kuchling : -- assignee: docs@python -> akuchling nosy: +akuchling ___ Python tracker ___ ___ Python-bugs-list

[issue25296] Simple End-of-life guide covering all unsupported versions

2018-09-15 Thread Mariatta Wijaya
Mariatta Wijaya added the comment: This is not being tracked in: https://github.com/python/pythondotorg/issues/1302 -- nosy: +Mariatta ___ Python tracker ___

[issue34656] memory exhaustion in Modules/_pickle.c:1393

2018-09-15 Thread Serhiy Storchaka
Serhiy Storchaka added the comment: >>> import pickletools >>> pickletools.dis(b'\x80\x04\x95\x1d\x00\x00\x00\x00\x00\x00\x00}\x94(\x8c\x03age\x94K\x17\x8c\x03jobr\x8c\x07student\x94u.') 0: \x80 PROTO 4 2: \x95 FRAME 29 11: }EMPTY_DICT 12: \x94 MEMOIZE(as 0)

[issue34579] test_embed.InitConfigTests fail on AIX

2018-09-15 Thread STINNER Victor
STINNER Victor added the comment: New changeset d2067318c79f66cfdabc5715a02d5fa5ff81 by Victor Stinner (Michael Felt) in branch 'master': bpo-34579: Fix test_embed DEFAULT_CON AIX (GH-9063) https://github.com/python/cpython/commit/d2067318c79f66cfdabc5715a02d5fa5ff81 --

[issue34663] Support POSIX_SPAWN_USEVFORK flag in posix_spawn

2018-09-15 Thread STINNER Victor
STINNER Victor added the comment: Gregory: on Linux, it does change the behavior. The parent blocks until the child is spawned. Python should let the developer decide to opt-in. Serhiy: IMHO it's better to make posix_spawn() as dumb as possible, a simple wrapper to the C call.

[issue26979] The danger of PyType_FromSpec()

2018-09-15 Thread Christian Tismer
Christian Tismer added the comment: > 1) If I understand correctly, this problem could be solved by > per-class C storage? Something like PyModuleDef.m_size / > PyModule_GetState, but for classes? No. To obtain sizeof(PyType_Type), we simply used the Python code type.__basicsize__ It is

[issue34692] ok

2018-09-15 Thread arul
New submission from arul : resloved On Sat, 15 Sep 2018 at 13:29, Serhiy Storchaka wrote: > > Change by Serhiy Storchaka : > > > -- > resolution: -> not a bug > stage: -> resolved > status: open -> closed > > ___ > Python tracker >

[issue34692] ok

2018-09-15 Thread Serhiy Storchaka
Change by Serhiy Storchaka : -- resolution: -> not a bug stage: -> resolved status: open -> closed ___ Python tracker ___ ___

[issue34692] ok

2018-09-15 Thread arul
Change by arul : -- nosy: arulpython priority: normal severity: normal status: open title: ok ___ Python tracker ___ ___

[issue25296] Simple End-of-life guide covering all unsupported versions

2018-09-15 Thread Andreas Lutro
Andreas Lutro added the comment: I see this has been closed, but the page on the devguide is still really hard to find. On the python IRC channel I often see questions about this and I always have a hard time finding the page with the correct information. If I google "python version

[issue34686] Add `-r`, as opposed to `-R` to Python core interpreter

2018-09-15 Thread Erwan Le Pape
Erwan Le Pape added the comment: Great! My only concern with that is marshalling of untrusted data at runtime (I know, you shouldn't do that) can become a much more expensive operation. Is there any internal use of marshal beyond .pycs used at runtime by the core interpreter that might be

[issue29341] Missing accepting path-like object in docstrings of os module functions

2018-09-15 Thread Karthikeyan Singaravelan
Change by Karthikeyan Singaravelan : -- nosy: +xtreak ___ Python tracker ___ ___ Python-bugs-list mailing list Unsubscribe: