[issue39837] Remove Azure Pipelines from GitHub PRs

hai shi added the comment: > Sadly (again), closing/reopening a PR re-runs all CIs. At the first run, GH > Action macOS job passed. At the second run, the "Tests" step of GH Action > macOS job, but I'm clueless with its logs: (...) Oh, I encountered the same trouble twice :(

[issue40405] asyncio.as_completed documentation misleading

Guido van Rossum added the comment: Please work this out in the PR between the two of you, Kyle and Bar. -- ___ Python tracker ___

[issue40405] asyncio.as_completed documentation misleading

Kyle Stanley added the comment: Alternatively, I think "can wait for" would also read better than "allows to wait for", if that is preferred over my above suggestion. -- ___ Python tracker

[issue39966] mock 3.9 bug: Wrapped objects without __bool__ raise exception

Karthikeyan Singaravelan added the comment: Thanks Avram for the report. I have reopened issue25597. Closing this as the regression change has been reverted for 3.9. -- resolution: -> fixed stage: patch review -> resolved status: open -> closed

[issue25597] unittest.mock does not wrap dunder methods (__getitem__ etc)

Karthikeyan Singaravelan added the comment: The change has been reverted as per issue39966. I am reopening this for further discussion. -- resolution: fixed -> stage: resolved -> status: closed -> open ___ Python tracker

[issue40405] asyncio.as_completed documentation misleading

Kyle Stanley added the comment: > based upon the order of completion I forgot to explain this in the above message. I think something that mentions "order of completion" should be included. Although it's implied in the name `as_completed`, to me it seems worthwhile to be fully explicit with

[issue40405] asyncio.as_completed documentation misleading

Kyle Stanley added the comment: Yury Selivanov wrote: > I'd suggest to change to: "Return an iterator of coroutines. Each coroutine > allows to wait for the earliest next result from the set of the remaining > awaitables." The first sentence looks good to me, but I'm not certain about the

[issue35113] inspect.getsource returns incorrect source for classes when class definition is part of multiline strings

Karthikeyan Singaravelan added the comment: Closing this as fixed with the enhancement to show decorator for classes too for 3.9. Thank you all for the help on this. -- resolution: -> fixed stage: patch review -> resolved status: open -> closed versions: +Python 3.9 -Python 3.6,

[issue40317] inspect.getsource() examines incorrect target

Karthikeyan Singaravelan added the comment: yes, thanks Ammar. Thanks Grzegorz for the report. -- resolution: -> duplicate stage: -> resolved status: open -> closed superseder: -> inspect.getsource returns incorrect source for classes when class definition is part of multiline

[issue40431] turtledemo/__main__.py: SyntaxError: invalid string prefix else"#fca"

Change by Kyle Stanley : -- resolution: -> fixed stage: patch review -> resolved status: open -> closed ___ Python tracker ___ ___

[issue40431] turtledemo/__main__.py: SyntaxError: invalid string prefix else"#fca"

Kyle Stanley added the comment: New changeset cc011b5190b63f0be561ddec38fc4cd9e60cbf6a by Kyle Stanley in branch '3.8': [3.8] bpo-40431: Fix syntax typo in turtledemo (GH-19777) (#19784) https://github.com/python/cpython/commit/cc011b5190b63f0be561ddec38fc4cd9e60cbf6a --

[issue39995] test_concurrent_futures: ProcessPoolSpawnExecutorDeadlockTest.test_crash() fails with OSError: [Errno 9] Bad file descriptor

Kyle Stanley added the comment: Also, thanks to Victor for coming up with an interim solution, and for the feedback from Antoine and Thomas. GH-19760 is a significant improvement from my initial proposal in GH-19751. -- ___ Python tracker

[issue39995] test_concurrent_futures: ProcessPoolSpawnExecutorDeadlockTest.test_crash() fails with OSError: [Errno 9] Bad file descriptor

Kyle Stanley added the comment: I am a bit concerned about the performance implication of accessing the thread wakeup through a lock in the call queue, but for now I think it's reasonable to address the buildbot failure with a locking solution while we try to find a better one. I'm not

[issue40433] Equality operations between lists and arrays

New submission from Ahmed Amr : Hi, I was thinking if we could add equality between array and list to work out of the box on the supported datatypes by arrays. Currently, comparing a list and an array with the same contents returns False. Also, creating an array of floats from a listA, then

[issue40428] [C API] Remove PyTuple_ClearFreeList() function (move it to the internal C API)

STINNER Victor added the comment: New changeset bb4a585d903e7fe0a46ded8c2ee3f47435ad6a66 by Zackery Spytz in branch 'master': bpo-40428: Remove references to Py*_ClearFreeList in the docs (GH-19783) https://github.com/python/cpython/commit/bb4a585d903e7fe0a46ded8c2ee3f47435ad6a66

[issue39995] test_concurrent_futures: ProcessPoolSpawnExecutorDeadlockTest.test_crash() fails with OSError: [Errno 9] Bad file descriptor

Change by Dong-hee Na : -- nosy: +corona10 ___ Python tracker ___ ___ Python-bugs-list mailing list Unsubscribe:

[issue40432] Pegen regenerate project for Windows not working

Change by anthony shaw : -- keywords: +patch pull_requests: +19108 stage: -> patch review pull_request: https://github.com/python/cpython/pull/19785 ___ Python tracker ___

[issue40432] Pegen regenerate project for Windows not working

New submission from anthony shaw : The additional tasks in the MSBuild project for pegen regeneration are not functional: - Setting PYTHONPATH= inline cannot be done in Windows using that method - The task does not inherit environment variables that way - The path to the peg_generator module

[issue40431] turtledemo/__main__.py: SyntaxError: invalid string prefix else"#fca"

Change by Kyle Stanley : -- pull_requests: +19107 pull_request: https://github.com/python/cpython/pull/19784 ___ Python tracker ___

[issue40431] turtledemo/__main__.py: SyntaxError: invalid string prefix else"#fca"

Change by Kyle Stanley : -- versions: -Python 3.7 ___ Python tracker ___ ___ Python-bugs-list mailing list Unsubscribe:

[issue40431] turtledemo/__main__.py: SyntaxError: invalid string prefix else"#fca"

Kyle Stanley added the comment: Victor Stinner wrote: > If you backport the change, please remove the NEWS entry, since it's not a > SyntaxError in 3.7 and 3.8. Good point. In that case, I'll do a manual backport of the PR just to 3.8 and skip the news entry. As mentioned above, I think

[issue40428] [C API] Remove PyTuple_ClearFreeList() function (move it to the internal C API)

Change by Zackery Spytz : -- nosy: +ZackerySpytz nosy_count: 1.0 -> 2.0 pull_requests: +19106 pull_request: https://github.com/python/cpython/pull/19783 ___ Python tracker ___

[issue40246] Different error messages for same error - invalid string prefixes

Kyle Stanley added the comment: For addressing the backwards compatibility concern, I think we should just convert it to a SyntaxWarning for cases like the above to indicate that it's not really correct syntax, but not harmful enough to justify code breakage. I think it fits the documented

[issue40334] PEP 617: new PEG-based parser

Pablo Galindo Salgado added the comment: New changeset 6d6508765514c7c10719478a0430f5e47c9a96ac by Lysandros Nikolaou in branch 'master': bpo-40334: Disallow invalid single statements in the new parser (GH-19774)

[issue40344] Programming FAQ about "What is the most efficient way to concatenate many strings together?" -- Improving the example

Change by Batuhan Taskaya : -- stage: patch review -> ___ Python tracker ___ ___ Python-bugs-list mailing list Unsubscribe:

[issue40344] Programming FAQ about "What is the most efficient way to concatenate many strings together?" -- Improving the example

Change by Batuhan Taskaya : -- keywords: -patch ___ Python tracker ___ ___ Python-bugs-list mailing list Unsubscribe:

[issue40344] Programming FAQ about "What is the most efficient way to concatenate many strings together?" -- Improving the example

Change by Batuhan Taskaya : -- pull_requests: -19105 ___ Python tracker ___ ___ Python-bugs-list mailing list Unsubscribe:

[issue40431] turtledemo/__main__.py: SyntaxError: invalid string prefix else"#fca"

STINNER Victor added the comment: If you backport the change, please remove the NEWS entry, since it's not a SyntaxError in 3.7 and 3.8. -- ___ Python tracker ___

[issue40334] PEP 617: new PEG-based parser

Change by STINNER Victor : -- nosy: -vstinner ___ Python tracker ___ ___ Python-bugs-list mailing list Unsubscribe:

[issue39995] test_concurrent_futures: ProcessPoolSpawnExecutorDeadlockTest.test_crash() fails with OSError: [Errno 9] Bad file descriptor

STINNER Victor added the comment: I'm still getting more and more buildbot emails about test_concurrent_futures, so I merged my PR 19760 to fix buildbots. Please revert or modify my PR 19760 if you have a better approach, but please check that test_killed_child() and

[issue40431] turtledemo/__main__.py: SyntaxError: invalid string prefix else"#fca"

Kyle Stanley added the comment: > I don't think that it's worth it to backport the change to 3.7 and 3.8: in > Python 3.7 and 3.8, the code is accepted. > Python 3.7 and 3.8 are not broken, so there is no need to fix them :-) I think we could very reasonably change `else"#fca"` -> `else

[issue40334] PEP 617: new PEG-based parser

Change by Batuhan Taskaya : -- pull_requests: +19104 pull_request: https://github.com/python/cpython/pull/19782 ___ Python tracker ___

[issue40344] Programming FAQ about "What is the most efficient way to concatenate many strings together?" -- Improving the example

Change by Batuhan Taskaya : -- pull_requests: +19105 pull_request: https://github.com/python/cpython/pull/19782 ___ Python tracker ___

[issue40344] Programming FAQ about "What is the most efficient way to concatenate many strings together?" -- Improving the example

Batuhan Taskaya added the comment: Sorry for the noise, wrong issue (thought 344, but actually was 334) -- ___ Python tracker ___

[issue40344] Programming FAQ about "What is the most efficient way to concatenate many strings together?" -- Improving the example

Change by Batuhan Taskaya : -- pull_requests: -19103 ___ Python tracker ___ ___ Python-bugs-list mailing list Unsubscribe:

[issue39995] test_concurrent_futures: ProcessPoolSpawnExecutorDeadlockTest.test_crash() fails with OSError: [Errno 9] Bad file descriptor

STINNER Victor added the comment: New changeset a4dfe8ede5a37576e17035dccfe109ba7752237e by Victor Stinner in branch 'master': bpo-39995: Fix concurrent.futures _ThreadWakeup (GH-19760) https://github.com/python/cpython/commit/a4dfe8ede5a37576e17035dccfe109ba7752237e --

[issue40344] Programming FAQ about "What is the most efficient way to concatenate many strings together?" -- Improving the example

Change by Batuhan Taskaya : -- keywords: +patch nosy: +BTaskaya nosy_count: 4.0 -> 5.0 pull_requests: +19103 stage: -> patch review pull_request: https://github.com/python/cpython/pull/19782 ___ Python tracker

[issue40421] [C API] Add getter functions for PyFrameObject and maybe move PyFrameObject to the internal C API

STINNER Victor added the comment: New changeset 703647732359200c54f1d2e695cc3a06b9a96c9a by Victor Stinner in branch 'master': bpo-40421: Add PyFrame_GetBack() function (GH-19765) https://github.com/python/cpython/commit/703647732359200c54f1d2e695cc3a06b9a96c9a --

[issue40275] test.support has way too many imports

STINNER Victor added the comment: New changeset 66abe98a816de84f89e2de4aa78cf09056227c25 by Hai Shi in branch 'master': bpo-40275: Move requires_hashdigest() to test.support.hashlib_helper (GH-19716) https://github.com/python/cpython/commit/66abe98a816de84f89e2de4aa78cf09056227c25

[issue40334] PEP 617: new PEG-based parser

Pablo Galindo Salgado added the comment: New changeset 2208134918ee673451e4fc525bbeab71142d794a by Pablo Galindo in branch 'master': bpo-40334: Explicitly cast to int in pegen.c to fix a compiler warning (GH-19779)

[issue40275] test.support has way too many imports

Change by hai shi : -- pull_requests: +19102 pull_request: https://github.com/python/cpython/pull/19761 ___ Python tracker ___ ___

[issue40429] [C API] PyThreadState_GetFrame() and PyFrame_GetCode() should return a strong refrence

STINNER Victor added the comment: New changeset 4386b9045e5fe1151e65c2816264b571eb9f by Victor Stinner in branch 'master': bpo-40429: PyThreadState_GetFrame() returns a strong ref (GH-19781) https://github.com/python/cpython/commit/4386b9045e5fe1151e65c2816264b571eb9f --

[issue39837] Remove Azure Pipelines from GitHub PRs

STINNER Victor added the comment: > Sadly (again), closing/reopening a PR re-runs all CIs. At the first run, GH > Action macOS job passed. At the second run, the "Tests" step of GH Action > macOS job, but I'm clueless with its logs: (...) Oops, I looked at two different PRs. In fact, the

[issue40246] Different error messages for same error - invalid string prefixes

Lysandros Nikolaou added the comment: A possible solution would be to only emit a SyntaxError if the NAME directly preceding a STRING token contains one of the valid string prefixes (either one of 'f', 'r', 'u', 'b'). This would still output a nicer error message, but would not break code

[issue40334] PEP 617: new PEG-based parser

Pablo Galindo Salgado added the comment: New changeset 37af21b667a9f41437b5b8e451497d7725016df5 by Lysandros Nikolaou in branch 'master': bpo-40334: Fix shifting of nested f-strings in the new parser (GH-19771) https://github.com/python/cpython/commit/37af21b667a9f41437b5b8e451497d7725016df5

[issue40429] [C API] PyThreadState_GetFrame() and PyFrame_GetCode() should return a strong refrence

Change by STINNER Victor : -- pull_requests: +19101 pull_request: https://github.com/python/cpython/pull/19781 ___ Python tracker ___

[issue40246] Different error messages for same error - invalid string prefixes

STINNER Victor added the comment: I think that a linter can be very pedantic on such code. My concern is about backward compatibility. The >else"#fca"< code was added 6 years ago (commit 8b95d5e0bf00d9d0098579d29fd6bb9322071879) and nobody complained, it "just worked". If we keep the

[issue40428] [C API] Remove PyTuple_ClearFreeList() function (move it to the internal C API)

STINNER Victor added the comment: Note: I created this issue while working on bpo-40421, when I moved PyFrame_ClearFreeList() definition from Include/frameobject.h to Include/cpython/frameobject.h. -- ___ Python tracker

[issue40428] [C API] Remove PyTuple_ClearFreeList() function (move it to the internal C API)

STINNER Victor added the comment: New changeset ae00a5a88534fd45939f86c12e038da9fa6f9ed6 by Victor Stinner in branch 'master': bpo-40428: Remove PyTuple_ClearFreeList() function (GH-19769) https://github.com/python/cpython/commit/ae00a5a88534fd45939f86c12e038da9fa6f9ed6 --

[issue40428] [C API] Remove PyTuple_ClearFreeList() function (move it to the internal C API)

Change by STINNER Victor : -- resolution: -> fixed stage: patch review -> resolved status: open -> closed ___ Python tracker ___

[issue40429] [C API] PyThreadState_GetFrame() and PyFrame_GetCode() should return a strong refrence

STINNER Victor added the comment: New changeset cc0dc7e484c9626857e9a8b4c40eee37473702ed by Victor Stinner in branch 'master': bpo-40429: Refactor super_init() (GH-19776) https://github.com/python/cpython/commit/cc0dc7e484c9626857e9a8b4c40eee37473702ed --

[issue39837] Remove Azure Pipelines from GitHub PRs

STINNER Victor added the comment: > Sadly, I had to close/reopen my PR since there is no button to only restart > the failure job, or even restart all Azure Pipeline jobs. This retrigger all > CI jobs :-( Sadly (again), closing/reopening a PR re-runs all CIs. At the first run, GH Action

[issue40246] Different error messages for same error - invalid string prefixes

Guido van Rossum added the comment: Personally I don't think that `else"stuff"` should become a syntax error. It's no different from other edge cases where for some reason the space seems optional, like `1and 2`. -- ___ Python tracker

[issue40246] Different error messages for same error - invalid string prefixes

STINNER Victor added the comment: > I personally think the new behaviour is more consistent and more likely to > catch errors. Would it be possible to emit a SyntaxWarning in 3.9 and only emit a SyntaxError in 3.10? -- ___ Python tracker

[issue40431] turtledemo/__main__.py: SyntaxError: invalid string prefix else"#fca"

STINNER Victor added the comment: > I am OK. I don't see why not to backport it, but I don't care much. Python 3.7 and 3.8 are not broken, so there is no need to fix them :-) Thanks for the fix. I reported the issue to: https://bugs.python.org/issue40246#msg367587 -- resolution:

[issue40246] Different error messages for same error - invalid string prefixes

Pablo Galindo Salgado added the comment: I personally think the new behaviour is more consistent and more likely to catch errors. -- ___ Python tracker ___

[issue38880] Subinterpreters: List interpreters associated with a channel end

miss-islington added the comment: New changeset f7bbf58aa9299e9dd00b7a1bdd1113b4dcb6dfdf by Lewis Gaul in branch 'master': bpo-38880: List interpreters associated with a channel end (GH-17323) https://github.com/python/cpython/commit/f7bbf58aa9299e9dd00b7a1bdd1113b4dcb6dfdf --

[issue40334] PEP 617: new PEG-based parser

Change by Guido van Rossum : -- pull_requests: +19100 pull_request: https://github.com/python/cpython/pull/19780 ___ Python tracker ___

[issue40246] Different error messages for same error - invalid string prefixes

STINNER Victor added the comment: I reopen the issue. The following code works on Python 3.8, but fails with SyntaxError on Python 3.9.0a6 with the old and the new parser (see bpo-40431): clear = "NORMAL" print(dict(state=clear, bg="#d00" if clear else"#fca")) Well, the code looks

[issue40246] Different error messages for same error - invalid string prefixes

Lysandros Nikolaou added the comment: On bpo-40334 @vstinner reported that this change broke some code in turtledemo. The code looks like this: print(dict(state=clear, bg="#d00" if clear else"#fca")) Due to the absence of a space between `else` and `"`, the else keyword is now interpreted

[issue40430] ast.Slice is no longer a subclass of ast.slice

Batuhan Taskaya added the comment: This has been discussee and rejected (for certain reasonsons): https://github.com/python/cpython/pull/19056#discussion_r396087689 -- nosy: +BTaskaya, serhiy.storchaka ___ Python tracker

[issue40431] turtledemo/__main__.py: SyntaxError: invalid string prefix else"#fca"

Miro Hrončok added the comment: I am OK. I don't see why not to backport it, but I don't care much. -- ___ Python tracker ___ ___

[issue40431] turtledemo/__main__.py: SyntaxError: invalid string prefix else"#fca"

STINNER Victor added the comment: I don't think that it's worth it to backport the change to 3.7 and 3.8: in Python 3.7 and 3.8, the code is accepted. I also reported the issue to: https://bugs.python.org/issue40334#msg367580 Miro: are you ok to close the issue (not backport)? --

[issue40431] turtledemo/__main__.py: SyntaxError: invalid string prefix else"#fca"

STINNER Victor added the comment: New changeset 49f70db83e2c62ad06805927f53f6c3e8f4b798e by Miro Hrončok in branch 'master': bpo-40431: Fix syntax typo in turtledemo (GH-19777) https://github.com/python/cpython/commit/49f70db83e2c62ad06805927f53f6c3e8f4b798e --

[issue40334] PEP 617: new PEG-based parser

Pablo Galindo Salgado added the comment: > The following code works on Python 3.8, but fails with SyntaxError on Python > 3.9.0a6 with the old and the new parser (see bpo-40431): This is not due to the new parser but due to https://bugs.python.org/issue40246. Please, report it there as

[issue40334] PEP 617: new PEG-based parser

STINNER Victor added the comment: The following code works on Python 3.8, but fails with SyntaxError on Python 3.9.0a6 with the old and the new parser (see bpo-40431): clear = "NORMAL" print(dict(state=clear, bg="#d00" if clear else"#fca")) Well, the code looks like a typo error...

[issue40334] PEP 617: new PEG-based parser

Change by Pablo Galindo Salgado : -- pull_requests: +19099 pull_request: https://github.com/python/cpython/pull/19779 ___ Python tracker ___

[issue40317] inspect.getsource() examines incorrect target

Ammar Askar added the comment: Did you mean to close this Karthik? -- nosy: +ammar2 ___ Python tracker ___ ___ Python-bugs-list

[issue39837] Remove Azure Pipelines from GitHub PRs

STINNER Victor added the comment: Another Azure Pipeline failure on my https://github.com/python/cpython/pull/19769 PR, it looks like a random networking failure. Sadly, I had to close/reopen my PR since there is no button to only restart the failure job, or even restart all Azure Pipeline

[issue40334] PEP 617: new PEG-based parser

STINNER Victor added the comment: It seems like building Python on Windows 64-bit emits a few warnings: D:\a\1\s\Parser\pegen\pegen.c(610,91): warning C4267: '=': conversion from 'size_t' to 'int', possible loss of data [D:\a\1\s\PCbuild\pythoncore.vcxproj]

[issue40334] PEP 617: new PEG-based parser

Pablo Galindo Salgado added the comment: > Pablo: Would you mind to increase ware-gentoo-x86 timeout? Ex: 30 min instead > of 15 min. Done: https://github.com/python/buildmaster-config/pull/190 -- ___ Python tracker

[issue40334] PEP 617: new PEG-based parser

Change by Lysandros Nikolaou : -- pull_requests: +19098 pull_request: https://github.com/python/cpython/pull/19778 ___ Python tracker ___

[issue40431] turtledemo/__main__.py: SyntaxError: invalid string prefix else"#fca"

Change by Miro Hrončok : -- keywords: +patch pull_requests: +19097 stage: -> patch review pull_request: https://github.com/python/cpython/pull/19777 ___ Python tracker ___

[issue40429] [C API] PyThreadState_GetFrame() and PyFrame_GetCode() should return a strong refrence

Change by STINNER Victor : -- pull_requests: +19096 pull_request: https://github.com/python/cpython/pull/19776 ___ Python tracker ___

[issue40431] turtledemo/__main__.py: SyntaxError: invalid string prefix else"#fca"

New submission from Miro Hrončok : With Python 3.9.0a6 we get the following syntax error when bytecompiling the standard library in Fedora: Compiling '/usr/lib64/python3.9/turtledemo/__main__.py'... *** File "/usr/lib64/python3.9/turtledemo/__main__.py", line 275 bg="#d00" if clear ==

[issue40429] [C API] PyThreadState_GetFrame() and PyFrame_GetCode() should return a strong refrence

STINNER Victor added the comment: New changeset 8852ad4208e34825f74e24945edb5bcf055d94fe by Victor Stinner in branch 'master': bpo-40429: PyFrame_GetCode() now returns a strong reference (GH-19773) https://github.com/python/cpython/commit/8852ad4208e34825f74e24945edb5bcf055d94fe --

[issue40334] PEP 617: new PEG-based parser

Change by Pablo Galindo Salgado : -- pull_requests: +19095 pull_request: https://github.com/python/cpython/pull/19775 ___ Python tracker ___

[issue32604] Expose the subinterpreters C-API in Python for testing use.

miss-islington added the comment: New changeset 5e8c691594d68925213d36296ce7c4b3e90bcb1d by Eric Snow in branch 'master': bpo-32604: Add support for a "default" arg in channel_recv(). (GH-19770) https://github.com/python/cpython/commit/5e8c691594d68925213d36296ce7c4b3e90bcb1d --

[issue40334] PEP 617: new PEG-based parser

Change by Lysandros Nikolaou : -- pull_requests: +19094 pull_request: https://github.com/python/cpython/pull/19774 ___ Python tracker ___

[issue40429] [C API] PyThreadState_GetFrame() and PyFrame_GetCode() should return a strong refrence

Change by STINNER Victor : -- pull_requests: +19093 pull_request: https://github.com/python/cpython/pull/19773 ___ Python tracker ___

[issue40429] [C API] PyThreadState_GetFrame() and PyFrame_GetCode() should return a strong refrence

STINNER Victor added the comment: New changeset 6d86a2331e6b64a2ae80c1a21f81baa5a71ac594 by Victor Stinner in branch 'master': bpo-40429: PyFrame_GetCode() result cannot be NULL (GH-19772) https://github.com/python/cpython/commit/6d86a2331e6b64a2ae80c1a21f81baa5a71ac594 --

[issue40430] ast.Slice is no longer a subclass of ast.slice

New submission from Anthony Sottile : unclear if this is intentional or not, I noticed this while seeing that `ast.Subscript.slice` is no longer `Index` / `Slice` / `ExtSlice` # python3.8 >>> isinstance(ast.Slice(), ast.slice) True # python3.9a6 >>> isinstance(ast.Slice(), ast.slice) False

[issue40429] [C API] PyThreadState_GetFrame() and PyFrame_GetCode() should return a strong refrence

Change by STINNER Victor : -- keywords: +patch pull_requests: +19092 stage: -> patch review pull_request: https://github.com/python/cpython/pull/19772 ___ Python tracker ___

[issue40429] [C API] PyThreadState_GetFrame() and PyFrame_GetCode() should return a strong refrence

New submission from STINNER Victor : I recently added PyThreadState_GetFrame() and PyFrame_GetCode() functions to the C API of Python 3.9. Currently, these functions return borrowed references. I asked on capi-sig and Brett confirms that borrowed refrences should be avoided:

[issue38787] PEP 573: Module State Access from C Extension Methods

Terry J. Reedy added the comment: The PEP is listed as accepted, with you as the BDFL-delegate, but it lacks a link to the public acceptance message (usually on pydev list). The PR was not rejected by a core dev but was closed without explanation by the author on the same day the last

[issue39117] Performance regression for making bound methods

STINNER Victor added the comment: By the way, in sorted(data, key=str.upper): str.upper is an unbound method, no? $ ./python Python 3.9.0a6+ (heads/master:d9a43e20fa, Apr 28 2020, 23:50:37) >>> type(str.upper) -- ___ Python tracker

[issue39117] Performance regression for making bound methods

STINNER Victor added the comment: I compared sorted(data, key=str.upper) performance between before the removal of the free list (parent of commit 3e54b575313c64f541e98216ed079fafed01ff5d) and the current master branch, I get: Mean +- std dev: [master] 167 us +- 4 us -> [cache] 179 us +- 7

[issue39117] Performance regression for making bound methods

STINNER Victor added the comment: > read_boundmethod 27.7 ns47.1 ns Extract of Tools/scripts/var_access_benchmark.py: def read_boundmethod(trials=trials, a=A()): for t in trials: a.m;a.m;a.m;a.m;a.m a.m;a.m;a.m;a.m;a.m

[issue40334] PEP 617: new PEG-based parser

Change by Lysandros Nikolaou : -- pull_requests: +19091 pull_request: https://github.com/python/cpython/pull/19771 ___ Python tracker ___

[issue32604] Expose the subinterpreters C-API in Python for testing use.

Change by Eric Snow : -- pull_requests: +19090 pull_request: https://github.com/python/cpython/pull/19770 ___ Python tracker ___

[issue37340] remove free_list for bound method objects

STINNER Victor added the comment: > sorted(data, key=str.upper) Oh, I guess that it's bpo-39117. -- ___ Python tracker ___ ___

[issue40334] PEP 617: new PEG-based parser

STINNER Victor added the comment: On x86 Gentoo Non-Debug with X 3.x buildbot, test_peg_generator timed out twice after 15 minutes (first time when tests are run in parallel, then when the test is run sequentially). Maybe the buildbot worker is just slow. On a previous build (749),

[issue37340] remove free_list for bound method objects

STINNER Victor added the comment: > This caused a performance regression (70%) for a fundamental operation. See > issue 37340. Which issue? This is bpo-37340. -- ___ Python tracker

[issue40428] [C API] Remove PyTuple_ClearFreeList() function (move it to the internal C API)

STINNER Victor added the comment: Only PyTuple_ClearFreeList() was exported by PC/python3.def. -- ___ Python tracker ___ ___

[issue40428] [C API] Remove PyTuple_ClearFreeList() function (move it to the internal C API)

Change by STINNER Victor : -- keywords: +patch pull_requests: +19089 stage: -> patch review pull_request: https://github.com/python/cpython/pull/19769 ___ Python tracker ___

[issue40428] [C API] Remove PyTuple_ClearFreeList() function (move it to the internal C API)

Change by STINNER Victor : -- title: [C API] Remove PyTuple_ClearFreeList() function -> [C API] Remove PyTuple_ClearFreeList() function (move it to the internal C API) ___ Python tracker

[issue40428] [C API] Remove PyTuple_ClearFreeList() function

Change by STINNER Victor : -- title: [C API] Move PyXXX_ClearFreeLists() functions to the internal C API -> [C API] Remove PyTuple_ClearFreeList() function ___ Python tracker

[issue40428] [C API] Move PyXXX_ClearFreeLists() functions to the internal C API

New submission from STINNER Victor : The following C functions are implementation details and should not be called directly: - PyAsyncGen_ClearFreeLists() - PyContext_ClearFreeList() - PyDict_ClearFreeList() - PyFloat_ClearFreeList() - PyFrame_ClearFreeList() - PyList_ClearFreeList() -

[issue40425] Refleak in CDataObject

Change by Joseph Sible : -- nosy: +Joseph Sible ___ Python tracker ___ ___ Python-bugs-list mailing list Unsubscribe:

[issue40390] Implement _xxsubinterpreters.channel_send_wait().

Eric Snow added the comment: Thanks, Ben. I'll take a look. -- components: +Library (Lib) -C API title: Implement a C API for channel_send_wait for subinterpreters. -> Implement _xxsubinterpreters.channel_send_wait(). ___ Python tracker

  1   2   >