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

Use Py_REFCNT instead of ->ob_refcnt #475

Merged
merged 2 commits into from
Mar 10, 2025

Conversation

pilleye
Copy link
Contributor

@pilleye pilleye commented Mar 10, 2025

Py_REFCNT was stabilized in 3.9, uses this official API instead of the ob_refcnt field that doesn't exist in the free-threaded build of 3.13.

Closes #464.

Py_REFCNT was stabilized in 3.9, uses this official API instead of the
`ob_refcnt` field that doesn't exist in the free-threaded build of 3.13.
@stefanseefeld
Copy link
Member

Please conditionalize the use of Py_REFCNT to allow for the code to work with older Python versions as well.

@pilleye
Copy link
Contributor Author

pilleye commented Mar 10, 2025

Fixed.

@stefanseefeld stefanseefeld merged commit 3e7be69 into boostorg:develop Mar 10, 2025
19 checks passed
@pilleye pilleye deleted the pilleye/py_refcnt branch March 10, 2025 20:35
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.

error C2039: 'ob_refcnt': is not a member of '_object'
2 participants