8000 gh-99767: update PyTypeObject docs for type watchers by carljm · Pull Request #99928 · python/cpython · GitHub
[go: up one dir, main page]
Skip to content

gh-99767: update PyTypeObject docs for type watchers #99928

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

Merged
merged 3 commits into from
Dec 15, 2022
Merged

gh-99767: update PyTypeObject docs for type watchers #99928

Changes from 1 commit
Commits
Show all changes
3 commits
Select commit Hold shift + click to select a range
970b03e
gh-99767: update PyTypeObject docs for type watchers
carljm Dec 1, 2022
53d33a1
Merge branch 'main' into typedocs
carljm Dec 14, 2022
b1a1157
Remove description of tp_watched, just say its internal
carljm Dec 14, 2022
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Remove description of tp_watched, just say its internal
  • Loading branch information
@carljm
carljm committed Dec 14, 2022
commit b1a1157fec2c23840b3117f385104da9273501bd
4 changes: 1 addition & 3 deletions Doc/c-api/typeobj.rst
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -2094,9 +2094,7 @@ and :c:type:`PyType_Type` effectively act as defaults.)

.. c:member:: char PyTypeObject.tp_watched
Copy link
Member

Choose a reason for hiding this comment

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

I would rather the docs say "Internal. Do not use."

TBH, I would like the whole of the type struct to be "do not use", but let's not make things any worse.

Copy link
Member Author

Choose a reason for hiding this comment

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

Are you wanting the existing text removed and replaced so it says only "Internal, do not use"? Or do you want to add "Internal, do not use" but also keep the description of what it is?

Copy link
Member Author

Choose a reason for hiding this comment

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

Pushed a version that removes the existing text, but I can easily re-add it.


Bitset marking which of the (up to eight) registered :c:func:`type
watchers<PyType_AddWatcher>` are watching this type (via
:c:func:`PyType_Watch`.)
Internal. Do not use.

.. versionadded:: 3.12

Expand Down
0