[issue35879] test_type_comments leaks references

Pablo Galindo Salgado added the comment: I'm almost sure this is not the correct fix, but I think these findings point to that object as the one being leaked. I do not understand how this object should be deallocated and where, I am trying to follow how the other PyObject* (name) is dealloca

[issue35879] test_type_comments leaks references

Guido van Rossum added the comment: I have been trying to do the same thing and likewise, no luck understanding yet how type comments are different here. -- ___ Python tracker __

[issue35879] test_type_comments leaks references

Guido van Rossum added the comment: In the meantime here's the fix for the '# type: ignore' leak. https://github.com/gvanrossum/cpython/commit/ef3b9e952906aab8f5452ebcf3b9e359d35615b4 -- ___ Python tracker

[issue35877] parenthesis is mandatory for named expressions in while statement

Emily Morehouse added the comment: New changeset d4fceaafb8e3f8700d9ec6ab37a51e903392f74f by Emily Morehouse (Xtreak) in branch 'master': bpo-35877: Make parenthesis optional for named expression in while statement (GH-11724) https://github.com/python/cpython/commit/d4fceaafb8e3f8700d9ec6ab3

[issue35877] parenthesis is mandatory for named expressions in while statement

Change by Emily Morehouse : -- pull_requests: +11608, 11609 ___ Python tracker ___ ___ Python-bugs-list mailing list Unsubscribe: h

[issue35877] parenthesis is mandatory for named expressions in while statement

Change by Emily Morehouse : -- pull_requests: +11608 ___ Python tracker ___ ___ Python-bugs-list mailing list Unsubscribe: https://

[issue35877] parenthesis is mandatory for named expressions in while statement

Change by Emily Morehouse : -- pull_requests: +11608, 11609, 11610 ___ Python tracker ___ ___ Python-bugs-list mailing list Unsubscr

[issue35877] parenthesis is mandatory for named expressions in while statement

Emily Morehouse added the comment: Thanks, Karthikeyan! I added an additional test to make sure this gets coverage. I'll close out this issue once tests pass and I can merge that. -- ___ Python tracker

[issue35861] test_named_expressions raises SyntaxWarning

Change by Emily Morehouse : -- resolution: -> fixed stage: patch review -> resolved status: open -> closed ___ Python tracker ___ _

[issue35877] parenthesis is mandatory for named expressions in while statement

Emily Morehouse added the comment: New changeset ac19081c26eaa7de3e6aabeb789ddc2e7cdd5b24 by Emily Morehouse in branch 'master': bpo-35877: Add test for while loop named expression without parentheses (GH-11726) https://github.com/python/cpython/commit/ac19081c26eaa7de3e6aabeb789ddc2e7cdd5b2

[issue35877] parenthesis is mandatory for named expressions in while statement

Change by Emily Morehouse : -- resolution: -> fixed stage: patch review -> resolved status: open -> closed ___ Python tracker ___ _

[issue35877] parenthesis is mandatory for named expressions in while statement

Karthikeyan Singaravelan added the comment: Thanks a lot Guido and Emily for guidance on this issue. Happy to see this for alpha release :) -- ___ Python tracker ___

[issue34886] subprocess.run throws exception when input and stdin are passed as kwargs

Change by Rémi Lapeyre : -- keywords: +patch, patch pull_requests: +11611, 11612 stage: -> patch review ___ Python tracker ___ ___ P

[issue34886] subprocess.run throws exception when input and stdin are passed as kwargs

Change by Rémi Lapeyre : -- keywords: +patch, patch, patch pull_requests: +11611, 11612, 11613 stage: -> patch review ___ Python tracker ___ ___

[issue34886] subprocess.run throws exception when input and stdin are passed as kwargs

Change by Rémi Lapeyre : -- keywords: +patch pull_requests: +11611 stage: -> patch review ___ Python tracker ___ ___ Python-bugs-lis

[issue34886] subprocess.run throws exception when input and stdin are passed as kwargs

Rémi Lapeyre added the comment: I opened a PR with @josh.r proposed change. -- nosy: +remi.lapeyre ___ Python tracker ___ ___ Python

[issue35885] configparser: indentation

New submission from mrs.red : The configparser module does not have an option for indentation. I would like to indent the keys by tabs. Maybe we could implement an option for that? I already have some example code for it. -- components: Library (Lib) messages: 334729 nosy: mrs.red prio

[issue35879] test_type_comments leaks references

Change by Guido van Rossum : -- keywords: +patch pull_requests: +11614 stage: -> patch review ___ Python tracker ___ ___ Python-bug

[issue35879] test_type_comments leaks references

Change by Guido van Rossum : -- keywords: +patch, patch pull_requests: +11614, 11615 stage: -> patch review ___ Python tracker ___

[issue35879] test_type_comments leaks references

Change by Guido van Rossum : -- keywords: +patch, patch, patch pull_requests: +11614, 11615, 11616 stage: -> patch review ___ Python tracker ___ __

[issue35879] test_type_comments leaks references

Guido van Rossum added the comment: Found it! You need to call PyArena_AddPyObject(). I racked my brain and finally found the example code I needed at the end of new_identifier(). Please review quickly so we won't ship 3.8.0a1 with a memory leak. -- _

[issue35879] test_type_comments leaks references

Guido van Rossum added the comment: New changeset d2b4c19d53f5f021fb1c7c32d48033a92ac4fe49 by Guido van Rossum in branch 'master': bpo-35879: Fix type comment leaks (GH-11728) https://github.com/python/cpython/commit/d2b4c19d53f5f021fb1c7c32d48033a92ac4fe49 -- _

[issue35879] test_type_comments leaks references

Change by Guido van Rossum : -- assignee: -> gvanrossum resolution: -> fixed stage: patch review -> resolved status: open -> closed type: -> behavior ___ Python tracker ___

[issue35880] math.sin has no backward error; this isn't documented

Change by Terry J. Reedy : -- stage: -> needs patch versions: -Python 3.4, Python 3.5, Python 3.6 ___ Python tracker ___ ___ Pytho

[issue17972] inspect module docs omits many functions

Change by kernc : -- nosy: +kernc ___ Python tracker ___ ___ Python-bugs-list mailing list Unsubscribe: https://mail.python.org/mai

[issue35883] Change invalid unicode characters to replacement characters in argv

Eryk Sun added the comment: In Unix, Python 3.6 decodes the char * command line arguments via mbstowcs. In Linux, I see the following misbehavior of mbstowcs when decoding an overlong UTF-8 sequence: >>> mbstowcs = ctypes.CDLL(None, use_errno=True).mbstowcs >>> arg = bytes(x + 128 fo

[issue24177] Add https?_proxy support to http.client

Change by Demian Brecht : -- keywords: +patch pull_requests: +11617 stage: needs patch -> patch review ___ Python tracker ___ ___ Py

[issue24177] Add https?_proxy support to http.client

Change by Demian Brecht : -- keywords: +patch, patch, patch pull_requests: +11617, 11618, 11619 stage: needs patch -> patch review ___ Python tracker ___ __

[issue24177] Add https?_proxy support to http.client

Change by Demian Brecht : -- keywords: +patch, patch pull_requests: +11617, 11618 stage: needs patch -> patch review ___ Python tracker ___

[issue35845] Can't read a F-contiguous memoryview in physical order

Change by Stefan Krah : -- keywords: +patch, patch, patch pull_requests: +11620, 11621, 11622 stage: needs patch -> patch review ___ Python tracker ___

[issue35845] Can't read a F-contiguous memoryview in physical order

Change by Stefan Krah : -- keywords: +patch, patch pull_requests: +11620, 11621 stage: needs patch -> patch review ___ Python tracker ___ __

[issue35845] Can't read a F-contiguous memoryview in physical order

Change by Stefan Krah : -- keywords: +patch pull_requests: +11620 stage: needs patch -> patch review ___ Python tracker ___ ___ Pyth

[issue35321] None _frozen_importlib.__spec__.origin attribute

Change by Barry A. Warsaw : -- versions: +Python 3.6 ___ Python tracker ___ ___ Python-bugs-list mailing list Unsubscribe: https://

[issue35886] Move PyInterpreterState into Include/internal/pycore_pystate.h

New submission from Eric Snow : In November Victor created the Include/cpython directory and moved a decent amount of public (but not limited) API there. This included the PyInterpreterState struct. I'd like to move it into the "internal" headers since it is somewhat coupled to the internal

[issue35839] Suggestion: Ignore sys.modules entries with no __spec__ attribute in find_spec

Change by Barry A. Warsaw : -- nosy: +barry ___ Python tracker ___ ___ Python-bugs-list mailing list Unsubscribe: https://mail.pyth

[issue35886] Move PyInterpreterState into Include/internal/pycore_pystate.h

Change by Eric Snow : -- keywords: +patch pull_requests: +11623 stage: needs patch -> patch review ___ Python tracker ___ ___ Python

[issue35886] Move PyInterpreterState into Include/internal/pycore_pystate.h

Change by Eric Snow : -- keywords: +patch, patch pull_requests: +11623, 11624 stage: needs patch -> patch review ___ Python tracker ___

[issue35886] Move PyInterpreterState into Include/internal/pycore_pystate.h

Change by Eric Snow : -- keywords: +patch, patch, patch pull_requests: +11623, 11624, 11625 stage: needs patch -> patch review ___ Python tracker ___ __

[issue35886] Move PyInterpreterState into Include/internal/pycore_pystate.h

Eric Snow added the comment: FWIW, I was hoping to the same for PyThreadState but it looks like 6 of its fields are exposed in "stable" header files via the following macros: # Include/object.h Py_TRASHCAN_SAFE_BEGIN Py_TRASHCAN_SAFE_END Include.ceval.h Py_EnterRecursiveCall Py_Le

[issue35887] Doc string for updating the frozen version of importlib in _bootstrap.py incorrect

New submission from Nina Zakharenko : In the process of creating a fix for issue #35321, I noticed what I believe to be a documentation omission. In Lib/importlib/_bootstrap.py the top level comment states that: # IMPORTANT: Whenever making changes to this module, be sure to run # a top-leve

[issue35537] use os.posix_spawn in subprocess

Change by Nina Zakharenko : -- pull_requests: +11626 ___ Python tracker ___ ___ Python-bugs-list mailing list Unsubscribe: https://

[issue35537] use os.posix_spawn in subprocess

Change by Nina Zakharenko : -- pull_requests: +11626, 11627 ___ Python tracker ___ ___ Python-bugs-list mailing list Unsubscribe: h

[issue35537] use os.posix_spawn in subprocess

Change by Nina Zakharenko : -- pull_requests: +11626, 11627, 11628 ___ Python tracker ___ ___ Python-bugs-list mailing list Unsubscr

[issue35321] None _frozen_importlib.__spec__.origin attribute

Change by Nina Zakharenko : -- keywords: +patch pull_requests: +11629 stage: test needed -> patch review ___ Python tracker ___ ___

[issue34235] PyArg_ParseTupleAndKeywords: support required keyword arguments

Michael Sullivan added the comment: How about adding another sigil that indicates that subsequent keyword-only arguments are required? So then your example becomes (using ` as a totally strawman option): PyArg_ParseTupleAndKeywords(args, kwds, "O|O$O`O", kwlist, &a, &b, &d, &c) It's a l

[issue35780] Recheck logic in the C version of the lru_cache()

Change by Raymond Hettinger : -- pull_requests: +11630 ___ Python tracker ___ ___ Python-bugs-list mailing list Unsubscribe: https:

[issue35780] Recheck logic in the C version of the lru_cache()

Change by Raymond Hettinger : -- pull_requests: +11630, 11631 ___ Python tracker ___ ___ Python-bugs-list mailing list Unsubscribe:

[issue35780] Recheck logic in the C version of the lru_cache()

Change by Raymond Hettinger : -- pull_requests: +11630, 11631, 11632 ___ Python tracker ___ ___ Python-bugs-list mailing list Unsubs

[issue35813] shared memory construct to avoid need for serialization between processes

Davin Potts added the comment: New changeset e5ef45b8f519a9be9965590e1a0a587ff584c180 by Davin Potts in branch 'master': bpo-35813: Added shared_memory submodule of multiprocessing. (#11664) https://github.com/python/cpython/commit/e5ef45b8f519a9be9965590e1a0a587ff584c180 -- __

[issue35888] ssl module - could not get the server certificate w/o completed handshake

New submission from Lee Eric : Hi, I'm not sure if this is the right place to ask after I exhausted several communication ways. I'm trying to use standard ssl module to get the server certificate details. If I understand correctly, the certificate I can get only when the TLS/SSL handshake is

<    1   2