Thanks to visit codestin.com
Credit goes to github.com

Skip to content

gh-116851: remove "from ctypes import *" from ctypes documents #116852

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.

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 1 commit into from
Mar 16, 2024
Merged

gh-116851: remove "from ctypes import *" from ctypes documents #116852

merged 1 commit into from
Mar 16, 2024

Conversation

jnchen
Copy link
Contributor

@jnchen jnchen commented Mar 15, 2024

remove import statement "from ctypes import *" from some code snippet before "libc", because "libc" cannot import ctypes, it cause ambiguous


📚 Documentation preview 📚: https://cpython-previews--116852.org.readthedocs.build/

remove import statement "from ctypes import *" from some code snippet before "libc", because "libc" cannot import ctypes, it cause ambiguous
@ghost
Copy link

ghost commented Mar 15, 2024

All commit authors signed the Contributor License Agreement.
CLA signed

@bedevere-app bedevere-app bot added docs Documentation in the Doc dir skip news labels Mar 15, 2024
@hugovk hugovk added needs backport to 3.11 only security fixes needs backport to 3.12 only security fixes labels Mar 15, 2024
@miss-islington-app
Copy link

Thanks @jnchen for the PR, and @serhiy-storchaka for merging it 🌮🎉.. I'm working now to backport this PR to: 3.11, 3.12.
🐍🍒⛏🤖

miss-islington pushed a commit to miss-islington/cpython that referenced this pull request Mar 16, 2024
…ythonGH-116852)

It is confusing, because libc is not imported from ctypes,
but defined in previous examples, which already contain the import.
(cherry picked from commit 744c077)

Co-authored-by: jnchen <[email protected]>
@bedevere-app
Copy link

bedevere-app bot commented Mar 16, 2024

GH-116905 is a backport of this pull request to the 3.12 branch.

@bedevere-app bedevere-app bot removed the needs backport to 3.12 only security fixes label Mar 16, 2024
miss-islington pushed a commit to miss-islington/cpython that referenced this pull request Mar 16, 2024
…ythonGH-116852)

It is confusing, because libc is not imported from ctypes,
but defined in previous examples, which already contain the import.
(cherry picked from commit 744c077)

Co-authored-by: jnchen <[email protected]>
@bedevere-app
Copy link

bedevere-app bot commented Mar 16, 2024

GH-116906 is a backport of this pull request to the 3.11 branch.

@bedevere-app bedevere-app bot removed the needs backport to 3.11 only security fixes label Mar 16, 2024
serhiy-storchaka pushed a commit that referenced this pull request Mar 16, 2024
…GH-116852) (GH-116905)

It is confusing, because libc is not imported from ctypes,
but defined in previous examples, which already contain the import.
(cherry picked from commit 744c077)

Co-authored-by: jnchen <[email protected]>
serhiy-storchaka pushed a commit that referenced this pull request Mar 16, 2024
…GH-116852) (GH-116906)

It is confusing, because libc is not imported from ctypes,
but defined in previous examples, which already contain the import.
(cherry picked from commit 744c077)

Co-authored-by: jnchen <[email protected]>
vstinner pushed a commit to vstinner/cpython that referenced this pull request Mar 20, 2024
…ythonGH-116852)

It is confusing, because libc is not imported from ctypes,
but defined in previous examples, which already contain the import.
adorilson pushed a commit to adorilson/cpython that referenced this pull request Mar 25, 2024
…ythonGH-116852)

It is confusing, because libc is not imported from ctypes,
but defined in previous examples, which already contain the import.
diegorusso pushed a commit to diegorusso/cpython that referenced this pull request Apr 17, 2024
…ythonGH-116852)

It is confusing, because libc is not imported from ctypes,
but defined in previous examples, which already contain the import.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
docs Documentation in the Doc dir skip news
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants