-
-
Notifications
You must be signed in to change notification settings - Fork 1.2k
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
Issue 9519 super init with non self arg #10190
base: main
Are you sure you want to change the base?
Issue 9519 super init with non self arg #10190
Conversation
the test file lines 11-13
Running
which is this:
so the function actually exits earlier than what was suggested in the original issue. |
Current debug cruft:
|
Just prior to turning off the debug cruft, here is the output:
|
c4f2c85
to
e29a7c8
Compare
There's a bunch of cruft in the commits and the code, but the idea is there, so I'm going to mark this as ready for review. I'm sure the actual ast checking for the |
As you can probably tell from my hacking, I'm very new at ast hacking, so there may be a far better way to do it than what I have. At least the test case should be good as a starting point. |
There are some test failures, but I'll wait for feedback/notes before I try to address them, as there will likely be changes needed. |
Work-in-progress for issue #9519, to see if I can crack it.
This is going to contain many junk commits that I'll rebase and get rid of later, just doing this so that people can see it and maybe comment.
The first commit adds a failing test case, then there is some debug crap just so I can understand the code as I go.