Skip to content

bpo-43181: Make PyObject_TypeCheck a static inline function#24533

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.

Sign up for GitHub

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

Jump to bottom
Merged
vstinner merged 5 commits into from
Feb 15, 2021
Merged

bpo-43181: Make PyObject_TypeCheck a static inline function #24533

vstinner merged 5 commits into from
Feb 15, 2021

Conversation

@erlend-aasland
Copy link
Contributor

@erlend-aaslanderlend-aasland commented Feb 15, 2021•
edited by bedevere-bot
Loading

@erlend-aasland
Copy link
ContributorAuthor

erlend-aasland commented Feb 15, 2021

@vstinner Would you mind reviewing this?

(cc. @gpshead )

vstinner
vstinner reviewed Feb 15, 2021
View reviewed changes
Erlend Egeberg Aaslandand others added 3 commits February 15, 2021 12:25
@vstinner
Improve NEWS text
36f742a
Co-authored-by: Victor Stinner <vstinner@python.org>
Address review
b19bf6a
- Improve naming: Use 'type' iso. 'tp' - Remove PyAPI_FUNC def - Remove unneeded parens
Fix ReST formatting
ad499ae
@erlend-aasland
Copy link
ContributorAuthor

erlend-aasland commented Feb 15, 2021

CI is done; PTAL @vstinner

@vstinnervstinner merged commit 4bb2a1e into python:masterFeb 15, 2021
@erlend-aaslanderlend-aasland deleted the make-pyobject-typecheck-inline branch February 15, 2021 16:22
@erlend-aaslanderlend-aasland mentioned this pull request Feb 16, 2021
Merged
erlend-aasland pushed a commit to erlend-aasland/cpython that referenced this pull request Feb 17, 2021
MovepythonGH-24533,pythonGH-24535, andpythonGH-24548NEWS entries …
c794f9c
…to "C API" section
adorilson pushed a commit to adorilson/cpython that referenced this pull request Mar 13, 2021
@adorilson
bpo-43181: Convert PyObject_TypeCheck to static inline function (pyth…
50002fc
…onGH-24533)
@numberZeronumberZeromannequin mentioned this pull request Jun 20, 2022
Closed
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@vstinnervstinnerAwaiting requested review from vstinner

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@erlend-aasland@vstinner@the-knights-who-say-ni@bedevere-bot