Skip to content

Add a warning about static nested class and magic browser dispatching#298

Open
Vampire wants to merge 1 commit into
apache:masterfrom
Vampire:static-nested-class-warning
Open

Add a warning about static nested class and magic browser dispatching#298
Vampire wants to merge 1 commit into
apache:masterfrom
Vampire:static-nested-class-warning

Conversation

@Vampire

@Vampire Vampire commented Dec 18, 2025

Copy link
Copy Markdown
Contributor

No description provided.

@cbmarcum

Copy link
Copy Markdown
Contributor

@Vampire I think this would be good to add after PR #297 is merged and the use of a static nested class would compile.

Just curious.. Is there a static nested class use-case where you ran into this Groovy issue?

@cbmarcum
cbmarcum self-requested a review December 27, 2025 22:11
@jonnybot0

Copy link
Copy Markdown
Contributor

Seems like my comment at #297 (comment) is relevant. I'd like to know a bit more about the "why" behind the change before we merge a change to the AST like the on in PR #297 that forwards errors to the runtime phase.

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants