Skip to content

Fix typo in exceptions.rst#150674

Open
manoj-marimuthu wants to merge 2 commits into
python:mainfrom
manoj-marimuthu:fix-typo-exceptions-rst
Open

Fix typo in exceptions.rst#150674
manoj-marimuthu wants to merge 2 commits into
python:mainfrom
manoj-marimuthu:fix-typo-exceptions-rst

Conversation

@manoj-marimuthu
Copy link
Copy Markdown
Contributor

This PR addresses minor typos in Doc/c-api/exceptions.rst by fixing the 2 following statements for better clarity and readability.

If *current_tstate* is not ``NULL`` then it will be used to identify what the
current thread is in the written output

and

This function is meant to debug debug situations such as segfaults, fatal
errors, and similar.

@bedevere-app bedevere-app Bot added docs Documentation in the Doc dir skip news awaiting review labels May 31, 2026
@github-project-automation github-project-automation Bot moved this to Todo in Docs PRs May 31, 2026
@manoj-marimuthu manoj-marimuthu changed the title Fix typo exceptions rst Fix typo in exceptions.rst May 31, 2026
@read-the-docs-community
Copy link
Copy Markdown

Documentation build overview

📚 cpython-previews | 🛠️ Build #32924060 | 📁 Comparing 0642177 against main (2f8f569)

  🔍 Preview build  

1 file changed
± c-api/exceptions.html

Comment thread Doc/c-api/exceptions.rst
error.

If *current_tstate* is not ``NULL`` then it will be used to identify what the
If *current_tstate* is not ``NULL`` then it will be used to identify which
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

This change is wrong, current_tstate is used to mark the current thread in the output, not to identify which thread is included in the output.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

@StanFromIreland so modifying it as "then it will be used to mark the current thread in the written output ...."
will be valid i guess ?

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

Labels

awaiting review docs Documentation in the Doc dir skip news

Projects

Status: Todo

Development

Successfully merging this pull request may close these issues.

2 participants