Skip to content

Conversation

serhiy-storchaka
Copy link
Member

the bare METH_FASTCALL be used for functions with positional-only
parameters.

@serhiy-storchaka serhiy-storchaka added the type-feature A feature request or enhancement label Jun 5, 2017
@mention-bot
Copy link

@serhiy-storchaka, thanks for your PR! By analyzing the history of the files in this pull request, we identified @zware, @larryhastings, @zooba and @tiran to be potential reviewers.

@serhiy-storchaka serhiy-storchaka force-pushed the fastcall-no-keywords branch 2 times, most recently from 3663631 to 823f887 Compare June 12, 2017 14:09
Copy link
Member

@vstinner vstinner left a comment

Choose a reason for hiding this comment

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

LGTM.

@vstinner
Copy link
Member

vstinner commented Jul 3, 2017

@serhiy-storchaka: Can you please rebase your change and then merge it?

the bare METH_FASTCALL be used for functions with positional-only
parameters.

Generated Argument Clinic code will be updated in the following commit.
@serhiy-storchaka serhiy-storchaka merged commit 6969eaf into python:master Jul 3, 2017
@serhiy-storchaka serhiy-storchaka deleted the fastcall-no-keywords branch July 3, 2017 18:20
@vstinner
Copy link
Member

vstinner commented Jul 3, 2017

Cool! Let's (micro)optimize this ;-)

vstinner added a commit that referenced this pull request Jul 5, 2017
vstinner added a commit that referenced this pull request Jul 5, 2017
* Revert "bpo-30854: Fix compile error when --without-threads (#2581)"

This reverts commit 0c31163.

* Revert "NEWS for 30777 (#2576)"

This reverts commit aaa917f.

* Revert "bpo-21624: IDLE -- minor htest fixes (#2575)"

This reverts commit 2000150.

* Revert "bpo-30777: IDLE: configdialog - add docstrings and improve comments (#2440)"

This reverts commit 7eb5883.

* Revert "bpo-30319: socket.close() now ignores ECONNRESET (#2565)"

This reverts commit 67e1478.

* Revert "bpo-30789: Use a single memory block for co_extra. (#2555)"

This reverts commit 378ebb6.

* Revert "bpo-30845: Enhance test_concurrent_futures cleanup (#2564)"

This reverts commit 3df9dec.

* Revert "bpo-29293: multiprocessing.Condition.notify() lacks parameter `n` (#2480)"

This reverts commit 4835041.

* Revert "Remove outdated FOX from GUI FAQ (GH-2538)"

This reverts commit d3ed287.

* Revert "bpo-6691: Pyclbr now reports nested classes and functions. (#2503)"

This reverts commit 246ff3b.

* Revert "bpo-29464: Rename METH_FASTCALL to METH_FASTCALL|METH_KEYWORDS and make (#1955)"

This reverts commit 6969eaf.

* Revert "bpo-30832: Remove own implementation for thread-local storage (#2537)"

This reverts commit aa0aa04.

* Revert "bpo-30764: Fix regrtest --fail-env-changed --forever (#2536)"

This reverts commit 5e87592.

* Revert "bpo-30822: Deduplicate ZoneInfoTest classes in test_datetime. (#2534)"

This reverts commit 34b5487.

* Revert "bpo-30822: Fix testing of datetime module. (#2530)"

This reverts commit 98b6bc3.
@rhettinger
Copy link
Contributor

This was a nice improvement. Thank you.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
type-feature A feature request or enhancement
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants