align varnames to CPython #6894
Conversation
|
Important Review skippedDraft detected. Please check the settings in the CodeRabbit UI or the You can disable this status message by setting the
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
|
Code has been automatically formatted The code in this PR has been formatted using:
git pull origin varnames |
| pub varnames: Box<[C::Name]>, | ||
| pub cellvars: Box<[C::Name]>, | ||
| pub freevars: Box<[C::Name]>, |
There was a problem hiding this comment.
@coolreader18 I remember you decided to keep this in current design before. I'd like to listen what do you think about the changes.
The motivation is aligning architecture more to CPython to reduce AI engineering cost. Because no actual technical advantage exists, I will keep our own way if you have good reasons not to do.
1 participant
No description provided.