Skip to content
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

bpo-29524: Add Objects/call.c file #12

Merged
merged 1 commit into from
Feb 12, 2017
Merged

bpo-29524: Add Objects/call.c file #12

merged 1 commit into from
Feb 12, 2017

Conversation

vstinner
Copy link
Member

@vstinner vstinner commented Feb 11, 2017

  • Move all functions to call objects in a new Objects/call.c file.
  • Rename fast_function() to _PyFunction_FastCallKeywords().
  • Copy null_error() from Objects/abstract.c
  • Inline type_error() in call.c to not have to copy it, it was only called once.
  • Export _PyEval_EvalCodeWithName() since it is now called from call.c.

http://bugs.python.org/issue29524

* Move all functions to call objects in a new Objects/call.c file.
* Rename fast_function() to _PyFunction_FastCallKeywords().
* Copy null_error() from Objects/abstract.c
* Inline type_error() in call.c to not have to copy it, it was only
  called once.
* Export _PyEval_EvalCodeWithName() since it is now called
  from call.c.
@vstinner vstinner added the performance Performance or resource usage label Feb 11, 2017
inlines the most primitive frame setup code from
PyEval_EvalCodeEx(), which vastly reduces the checks that must be
done before evaluating the frame.
*/
Copy link
Member

Choose a reason for hiding this comment

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

This comment block should be moved to function_code_fastcall().

@vstinner
Copy link
Member Author

vstinner commented Feb 11, 2017 via email

Copy link
Member

@methane methane left a comment

Choose a reason for hiding this comment

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

I just checked all code is moved as-is.
I agree the comment block is not so important.

@vstinner
Copy link
Member Author

vstinner commented Feb 12, 2017 via email

@vstinner vstinner merged commit c22bfaa into python:master Feb 12, 2017
@vstinner vstinner deleted the call_file branch June 15, 2017 23:02
@jdemeyer
Copy link
Contributor

Follow-up at #7909

paulmon added a commit to paulmon/cpython that referenced this pull request Jan 10, 2019
Windows arm32 - fix failing tests or skip
emmatyping added a commit to emmatyping/cpython that referenced this pull request Mar 16, 2020
isidentical referenced this pull request in isidentical/cpython Jun 29, 2021
* Handle lines with multi-byte unicode characters properly

* Use Py_XDECREF instead of Py_DECREF

Co-authored-by: Batuhan Taskaya <isidentical@gmail.com>

Co-authored-by: Batuhan Taskaya <isidentical@gmail.com>
jaraco pushed a commit that referenced this pull request Dec 2, 2022
kumaraditya303 pushed a commit that referenced this pull request Oct 14, 2024
…cycles (#12… (#125476)

Revert "gh-124958: fix asyncio.TaskGroup and _PyFuture refcycles (#124959)"

This reverts commit d5dbbf4.
miss-islington pushed a commit to miss-islington/cpython that referenced this pull request Oct 14, 2024
…PyFuture refcycles (pythonGH-12… (pythonGH-125476)

Revert "pythongh-124958: fix asyncio.TaskGroup and _PyFuture refcycles (pythonGH-124959)"

This reverts commit d5dbbf4.
(cherry picked from commit e99650b)

Co-authored-by: Kirill Podoprigora <kirill.bast9@mail.ru>
1st1 pushed a commit that referenced this pull request Oct 14, 2024
…p and _PyFuture refcycles ... (#125486)

* Revert "gh-125472: Revert "gh-124958: fix asyncio.TaskGroup and _PyFuture refcycles (#12… (#125476)"

This reverts commit e99650b.

* fix incompatability with gh-124392
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