Skip to content

Document why functools.partial() must copy kwargs #253

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 1 commit into from
Feb 23, 2017
Merged

Document why functools.partial() must copy kwargs #253

merged 1 commit into from
Feb 23, 2017

Conversation

vstinner
Copy link
Member

@vstinner vstinner commented Feb 23, 2017

Add a comment to prevent further attempts to avoid a copy for optimization.

http://bugs.python.org/issue27840

Add a comment to prevent further attempts to avoid a copy for
optimization.
@vstinner vstinner added the performance Performance or resource usage label Feb 23, 2017
Copy link
Member

@pitrou pitrou left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, thanks.

@vstinner vstinner merged commit 561ca80 into python:master Feb 23, 2017
@vstinner vstinner deleted the bpo-27840 branch February 27, 2017 13:44
akruis pushed a commit to akruis/cpython that referenced this pull request May 27, 2021
…3.8.0b1

Disable the support for Cython versions below 0.29.
Starting with Stackless version 3.8.0b1 extension modules compiled with
regular C-Python and Cython versions below 0.29 no longer work with
Stackless Python.
jaraco pushed a commit that referenced this pull request Dec 2, 2022
This also adds the "Automerge-triggered-by: @{coredev}" message at the end of the PR body.

Closes python/core-workflow#340
jaraco added a commit to jaraco/cpython that referenced this pull request Feb 17, 2023
jaraco added a commit to jaraco/cpython that referenced this pull request Feb 17, 2023
jaraco added a commit to jaraco/cpython that referenced this pull request Feb 17, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
performance Performance or resource usage
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants