| FazBrowse GitHub Viewer | Trending | | Home |
| Tools: [Download Repo ZIP] [Original HTTPS Page] |
Sorry, something went wrong.
|
Is the PR a draft or is it ready for review? |
Sorry, something went wrong.
This one is a draft. Just an example of concept that works for current functools.partial. Reason being that #119827 will need a different implementation, which I have, but still waiting for reply how to go about it (#119827 (comment)): Here is implementation that works for partial with Placeholders: It can be reviewed - sooner or later it will become PR. It is contained within one function - no documentation updates or anything additional is needed. Let me know if there is a good way to issue a separate PR on top of the one which is not yet merged - I am not aware of such. |
Sorry, something went wrong.
|
Will issue a new PR for partial with placeholders |
Sorry, something went wrong.
| Back | FazBrowse Home | New Git URL |
Although comment stated that "merging keyword arguments for vector call is messy" (not exact quote), but I have found that it is fairly straight forward.
This achieves 2 goals:
Number 1. is solved better than it was in #119125 as there are no more switching between methods and faster vectorcall is used whenever appropriate. Also, this does not require additional variable in class struct.
Comparison of performance:
This is only initial attempt and I think the code can be made simpler. (much simpler if decide to use _PyObject_VectorcallDictTstate).
This will need to be adjusted to #119827 so do not merge.