Movatterモバイル変換


[0]ホーム

URL:


Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Sign up
Appearance settings

gh-103092: Isolate _ctypes, part 1#103893

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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.

Already on GitHub?Sign in to your account

Merged

Conversation

erlend-aasland
Copy link
Contributor

@erlend-aaslanderlend-aasland commentedApr 26, 2023
edited by bedevere-bot
Loading

Low-hanging fruit: port the easy types

@erlend-aaslanderlend-aasland added the 🔨 test-with-refleak-buildbotsTest PR w/ refleak buildbots; report in status section labelApr 27, 2023
@bedevere-bot
Copy link

🤖 New build scheduled with the buildbot fleet by@erlend-aasland for commita276753 🤖

If you want to schedule another build, you need to add the🔨 test-with-refleak-buildbots label again.

@bedevere-botbedevere-bot removed the 🔨 test-with-refleak-buildbotsTest PR w/ refleak buildbots; report in status section labelApr 27, 2023
@erlend-aaslanderlend-aasland merged commite9c7772 intopython:mainApr 27, 2023
@erlend-aaslanderlend-aasland deleted the isolate-ctypes/part1 branchApril 27, 2023 12:57
@swt2c
Copy link

Hi@erlend-aasland, as part of this, it seems thatCArgObject moved frombuiltins to_ctypes. Just confirming that was intentional?

3.11:

>>> import ctypes>>> type(ctypes.byref(ctypes.c_int(0)))<class 'CArgObject'>

3.12

>>> import ctypes>>> type(ctypes.byref(ctypes.c_int(0)))<class '_ctypes.CArgObject'>

@erlend-aasland
Copy link
ContributorAuthor

erlend-aasland commentedJul 18, 2023
edited
Loading

CArgsObject still belongs toctypes, just as it does in 3.11. What changed was its fully qualified name only. That change is deliberate because it is needed to be able to look up the module the heap type belongs to.

EDIT: Thinking about it, I think the last sentence I wrote above is not right; I'm pretty sure module lookup would work anyways (but I don't intend to pursue this right now).

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

@kumaraditya303kumaraditya303kumaraditya303 approved these changes

@corona10corona10Awaiting requested review from corona10

Assignees
No one assigned
Labels
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

4 participants
@erlend-aasland@bedevere-bot@swt2c@kumaraditya303

[8]ページ先頭

©2009-2025 Movatter.jp