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.

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

Add syntactic support for T<string>::class #53

Merged
merged 3 commits into from
Aug 4, 2024

Conversation

thekid
Copy link
Member

@thekid thekid commented Aug 2, 2024

Example (when using https://github.com/xp-lang/xp-generics):

$name= T<string>::class; // "T\xb7\xb7\xfestring"

@thekid thekid changed the title Add syntactic support for `A<T>::class Add syntactic support for T<string>::class Aug 3, 2024
@thekid
Copy link
Member Author

thekid commented Aug 4, 2024

It would be great to normalize the various type members to all contain lang.ast.Type instances instead of using strings in certain situations. However, this would be a BC break and is thus out of scope for this PR.

@thekid thekid merged commit 9481a4e into master Aug 4, 2024
12 checks passed
thekid added a commit to xp-lang/xp-generics that referenced this pull request Aug 4, 2024
@thekid thekid deleted the feature/generic-class-resolution branch August 4, 2024 18:11
@thekid
Copy link
Member Author

thekid commented Aug 4, 2024

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant