Skip to content

Conversation

@guitargeek
Copy link
Contributor

@guitargeek guitargeek commented Mar 11, 2024

Also fix the typos found with:
https://github.com/crate-ci/typos

Validated by the ROOT sync PR:
root-project/root#14507

@wlav wlav merged commit 63c1d53 into wlav:master Mar 11, 2024
@wlav
Copy link
Owner

wlav commented Mar 11, 2024

I wasn't aware of the existence of the tp_vectorcall field ... I wrote that code for the py3.8 sources, where it isn't advertised (even though available). Currently, all CPPOverload objects have an fVectorCall field (as was per the documentation at the time). This is always set to mp_vectorcall. It'd be nice to remove that and safe a bit of memory.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants