Skip to content

WIP: gh-106593: PyResource API [DO NOT MERGE] - #106596

Closed
vstinner wants to merge 1 commit into
python:mainfrom
vstinner:seq_asobjarray
Closed

WIP: gh-106593: PyResource API [DO NOT MERGE]#106596
vstinner wants to merge 1 commit into
python:mainfrom
vstinner:seq_asobjarray

Conversation

@vstinner

@vstinner vstinner commented Jul 10, 2023 •
edited by github-actions Bot
Loading

Copy link
Copy Markdown
Member
  • Add PyUnicode_AsUTF8Resource()
  • Add PyBytes_AsStringResource()
  • Add PySequence_AsObjectArray()
  • Add Include/pyresource.h
  • Add PyResource_Release() to the stable ABI
  • compute_abstract_methods(): Replace PySequence_Fast() with PySequence_AsObjectArray()

📚 Documentation preview 📚: https://cpython-previews--106596.org.readthedocs.build/

@bedevere-bot bedevere-bot mentioned this pull request Jul 10, 2023
Closed
@vstinner

vstinner commented Jul 10, 2023

Copy link
Copy Markdown
Member Author

Proof-of-Concept PR for issue #106592 PyResource API and issue #106593 PyObject_AsObjectArray() API.

If these issues design is accepted, I will write a clean PR with doc and tests.

@vstinner vstinner mentioned this pull request Jul 10, 2023
Closed
encukou
encukou reviewed Jul 10, 2023
View reviewed changes
Comment thread Misc/stable_abi.toml
@vstinner
WIP: pythongh-106593: PyResource API
98d883b 
* Add PyUnicode_AsUTF8Resource()
* Add PyBytes_AsStringResource()
* Add PySequence_AsObjectArray()
* Add Include/pyresource.h
* Add PyResource_Close() to the stable ABI
* compute_abstract_methods(): Replace PySequence_Fast()
  with PySequence_AsObjectArray()
@vstinner

vstinner commented Aug 26, 2023

Copy link
Copy Markdown
Member Author

I close my PR: #106593 (comment)

@vstinner vstinner closed this Aug 26, 2023
@vstinner
vstinner deleted the seq_asobjarray branch August 26, 2023 02:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@encukou encukou encukou left review comments
@markshannon markshannon Awaiting requested review from markshannon markshannon will be requested when the pull request is marked ready for review markshannon is a code owner
@ZeroIntensity ZeroIntensity Awaiting requested review from ZeroIntensity ZeroIntensity will be requested when the pull request is marked ready for review ZeroIntensity is a code owner
@erlend-aasland erlend-aasland Awaiting requested review from erlend-aasland erlend-aasland will be requested when the pull request is marked ready for review erlend-aasland is a code owner
@AA-Turner AA-Turner Awaiting requested review from AA-Turner AA-Turner will be requested when the pull request is marked ready for review AA-Turner is a code owner
@emmatyping emmatyping Awaiting requested review from emmatyping emmatyping will be requested when the pull request is marked ready for review emmatyping is a code owner
@itamaro itamaro Awaiting requested review from itamaro itamaro will be requested when the pull request is marked ready for review itamaro is a code owner

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@vstinner @encukou