Skip to content

Make the crash warning in ctypes more prominent#150410

Open
StanFromIreland wants to merge 3 commits into
python:mainfrom
StanFromIreland:ctypes-warning
Open

Make the crash warning in ctypes more prominent#150410
StanFromIreland wants to merge 3 commits into
python:mainfrom
StanFromIreland:ctypes-warning

Conversation

@StanFromIreland
Copy link
Copy Markdown
Member

(I was in a warning-y mood today ;-)

I agree with Jelle:

The note you cite in the ctypes docs is buried in a section about a more specific topic (calling functions). I think it's worth adding a more prominent warning like the one on https://docs.python.org/3/library/pickle.html . However, once we do that I don't think we need to do anything in the devguide: the note in the CPython docs should be enough.

Originally posted by @JelleZijlstra in #211

Comment thread Doc/library/ctypes.rst Outdated
@read-the-docs-community
Copy link
Copy Markdown

read-the-docs-community Bot commented May 25, 2026

Documentation build overview

📚 cpython-previews | 🛠️ Build #32852780 | 📁 Comparing 0b1a090 against main (c714b56)

  🔍 Preview build  

2 files changed
± library/ctypes.html
± whatsnew/changelog.html

Copy link
Copy Markdown
Member

@JelleZijlstra JelleZijlstra left a comment

Choose a reason for hiding this comment

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

I like this wording! Please leave this open for a little bit to give others a chance to comment though.

Comment thread Doc/library/ctypes.rst Outdated
Co-authored-by: Petr Viktorin <encukou@gmail.com>
@StanFromIreland StanFromIreland requested a review from encukou May 26, 2026 12:51
Copy link
Copy Markdown
Contributor

@ncoghlan ncoghlan left a comment

Choose a reason for hiding this comment

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

Looks good to me, too!

@JelleZijlstra JelleZijlstra added needs backport to 3.13 bugs and security fixes needs backport to 3.14 bugs and security fixes needs backport to 3.15 pre-release feature fixes, bugs and security fixes labels May 27, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

awaiting merge docs Documentation in the Doc dir needs backport to 3.13 bugs and security fixes needs backport to 3.14 bugs and security fixes needs backport to 3.15 pre-release feature fixes, bugs and security fixes skip issue skip news topic-ctypes

Projects

Status: No status
Status: Todo

Development

Successfully merging this pull request may close these issues.

4 participants