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

Ignore types in macro runtime dependencies #15529

Merged
merged 1 commit into from
Jul 4, 2022

Conversation

nicolasstucki
Copy link
Contributor

@nicolasstucki nicolasstucki commented Jun 27, 2022

We can have different kinds of type references in the contents of a
splice. We do not care about types because those are erased and the
interpreter emulates erased semantics. These types are in Trees that
extend TypeTree or RefTree referring to type.

Fixes #12498

@nicolasstucki nicolasstucki marked this pull request as ready for review June 27, 2022 09:22
@nicolasstucki
Copy link
Contributor Author

How I fixed it: #12498 (comment)

Copy link
Member

@dwijnand dwijnand left a comment

Choose a reason for hiding this comment

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

💪🏼

@nicolasstucki nicolasstucki force-pushed the fix-#12498 branch 2 times, most recently from 0b6d5b5 to 6636a6d Compare July 4, 2022 14:25
@nicolasstucki
Copy link
Contributor Author

Squashed and rebased

@nicolasstucki nicolasstucki changed the title Ignore type parameters in macro runtime dependencies Ignore types in macro runtime dependencies Jul 4, 2022
We can have different kinds of type references in the contents of a
splice. We do not care about types because those are erased and the
interpreter emulates erased semantics. These types are in `Tree`s that
extend `TypeTree` or `RefTree` referring to type.

Fixes scala#12498
@nicolasstucki
Copy link
Contributor Author

@dwijnand something broke when I rebased. Now it is fixed. Could you double-check the code?

@nicolasstucki nicolasstucki merged commit dc5a9a4 into scala:main Jul 4, 2022
@nicolasstucki nicolasstucki deleted the fix-#12498 branch July 4, 2022 16:35
@Kordyjan Kordyjan added this to the 3.2.1 milestone Aug 1, 2023
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.

Cannot call macro defined in the same source file error
3 participants