Skip to content

Conversation

@vstinner
Copy link
Member

@vstinner vstinner commented Nov 5, 2025

  • 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
Member Author

vstinner commented Nov 5, 2025

cc @ZeroIntensity @picnixz

@vstinner vstinner added type-feature A feature request or enhancement topic-C-API labels Nov 5, 2025
@vstinner vstinner removed the type-feature A feature request or enhancement label Nov 5, 2025
@ZeroIntensity
Copy link
Member

The tests seem to be failing on Windows.

@vstinner
Copy link
Member Author

vstinner commented Nov 5, 2025

The tests seem to be failing on Windows.

object address : 0000017EAEAFA4D0

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

Copy link
Member

@ZeroIntensity ZeroIntensity 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.

vstinner and others added 3 commits November 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

encukou commented Nov 6, 2025

I think this is a great candidate for unstable API.

vstinner and others added 4 commits November 6, 2025 12:07
Co-authored-by: Kumar Aditya <kumaraditya@python.org>
Co-authored-by: Petr Viktorin <encukou@gmail.com>
@vstinner
Copy link
Member Author

vstinner commented Nov 6, 2025

I created a C API Working Group decision issue.

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants