Movatterモバイル変換

[0]ホーム
URL:

Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

 Sign up
Appearance settings

gh-141070: Add PyObject_Dump() function#141072

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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.

Already on GitHub?Sign in to your account

Open
vstinner wants to merge11 commits intopython:main
base:main
Choose a base branch
Loading
fromvstinner:object_dump

Conversation

@vstinner
Copy link
Member

@vstinnervstinner commentedNov 5, 2025•
edited by github-actionsbot
Loading

  • Promote _PyObject_Dump() as a public function.
  • Keep _PyObject_Dump() alias to PyObject_Dump() for backward compatibility.
  • Replace _PyObject_Dump() with PyObject_Dump().

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

* Promote _PyObject_Dump() as a public function.* Keep _PyObject_Dump() alias to PyObject_Dump() for backward  compatibility.* Replace _PyObject_Dump() with PyObject_Dump().
@vstinner
Copy link
MemberAuthor

cc@ZeroIntensity@picnixz

@vstinnervstinner added type-featureA feature request or enhancement topic-C-API labelsNov 5, 2025
@vstinnervstinner removed the type-featureA feature request or enhancement labelNov 5, 2025
@ZeroIntensity
Copy link
Member

The tests seem to be failing on Windows.

@vstinner
Copy link
MemberAuthor

The tests seem to be failing on Windows.

object address : 0000017EAEAFA4D0

Ah yes,printf("%p", ptr) doesn't print0x prefix on Windows.

Copy link
Member

@ZeroIntensityZeroIntensity left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others.Learn more.

LGTM with a few English nitpicks on the docs.

vstinnerand others added3 commitsNovember 6, 2025 00:18
Co-authored-by: Peter Bierma <zintensitydev@gmail.com>
Co-authored-by: Peter Bierma <zintensitydev@gmail.com>
Co-authored-by: Peter Bierma <zintensitydev@gmail.com>
@encukou
Copy link
Member

I think this is a great candidate for unstable API.

vstinnerand others added4 commitsNovember 6, 2025 12:07
Co-authored-by: Kumar Aditya <kumaraditya@python.org>
Co-authored-by: Petr Viktorin <encukou@gmail.com>
@vstinner
Copy link
MemberAuthor

I created aC API Working Group decision issue.

@ZeroIntensity
Copy link
Member

Hm,@encukou, why make this an unstable API? I don't think we ever plan to remove the ability to print objects from the C API. If you're concerned about people relying on the exact output, we can just tell them "don't do that" in the docs.

picnixz reacted with thumbs up emoji

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

Reviewers

@encukouencukouencukou left review comments

@ZeroIntensityZeroIntensityZeroIntensity approved these changes

@kumaraditya303kumaraditya303kumaraditya303 approved these changes

@ericsnowcurrentlyericsnowcurrentlyAwaiting requested review from ericsnowcurrentlyericsnowcurrently is a code owner

@AA-TurnerAA-TurnerAwaiting requested review from AA-TurnerAA-Turner is a code owner

Assignees

No one assigned

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@vstinner@ZeroIntensity@encukou@kumaraditya303
[8]ページ先頭
©2009-2025 Movatter.jp