Skip to content
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.

Sign up for GitHub

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

Jump to bottom

[typer] VUser arguments and variables named this #8712

Open
Gama11 opened this issue Aug 28, 2019 · 2 comments
Open

[typer] VUser arguments and variables named this #8712

Gama11 opened this issue Aug 28, 2019 · 2 comments
Assignees
@Simn
Labels
test-needed
Milestone
Later

Comments

@Gama11
Copy link
Member

Gama11 commented Aug 28, 2019

During #8708 I found that there's VUser locals from abstracts named this (both in type_vars and type_function), which required an exception to be made in the local variable name checks.

These should use VGenerated or some other new kind.
@Gama11 Gama11 added this to the Release 4.1 milestone Aug 28, 2019
@Simn Simn self-assigned this Feb 19, 2020
@Simn Simn modified the milestones: Release 4.1, Release 4.2 Feb 20, 2020
Simn added a commit that referenced this issue Mar 26, 2023
@Simn
[typer] use distinct tvar_kind for abstract this
e856e79 
see #8712
@Simn Simn modified the milestones: Release 4.3, Later Mar 26, 2023
@Simn
Copy link
Member

Simn commented Mar 26, 2023

The linked commit should address this, but I forget if we have a way to test it.

@Simn
Copy link
Member

Simn commented Mar 30, 2023

I noticed in #10747 that this wasn't working for function arguments and addressed it in the linked commit there.

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

@Simn Simn

Labels
test-needed
Projects
None yet
Milestone
Later
Development

No branches or pull requests
3 participants
@Simn @Gama11 @RealyUniqueName