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
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

Status: No status
Status: Todo

Development

Successfully merging this pull request may close these issues.

3 participants