Skip to content

Conversation

@iritkatriel
Copy link
Member

@iritkatrieliritkatriel commented Jul 2, 2024

Copy link
Contributor

@AraHaanAraHaan left a comment

Choose a reason for hiding this comment

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

I agree, this could be a good performance improvement when using a ton of c extensions with python scripts that use import from statements on them.

Co-authored-by: Bénédikt Tran <[email protected]>
@iritkatriel
Copy link
MemberAuthor

I agree, this could be a good performance improvement when using a ton of c extensions with python scripts that use import from statements on them.

This shouldn't impact performance.

This is about simplifying code in compile.c (in this case, making compiler_from_import no longer access the internals of struct compiler). This is part of a longer term plan to break up the compiler.

Copy link
Member

@JelleZijlstraJelleZijlstra 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. This is thoroughly tested in test_future_stmt/test_future.py and I confirmed that the code in future.c that sets ff_location makes sense.

@iritkatrieliritkatriel enabled auto-merge (squash) July 2, 2024 16:14
@iritkatrieliritkatriel merged commit 1ac2732 into python:mainJul 2, 2024
Akasurde pushed a commit to Akasurde/cpython that referenced this pull request Jul 3, 2024
noahbkim pushed a commit to hudson-trading/cpython that referenced this pull request Jul 11, 2024
estyxx pushed a commit to estyxx/cpython that referenced this pull request Jul 17, 2024
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants

@iritkatriel@JelleZijlstra@picnixz@AraHaan