Skip to content

Conversation

Ladicek
Copy link
Contributor

@Ladicek Ladicek commented Aug 5, 2025

  • improve message-less IllegalStateException in default switch cases
  • improve exception message in TryCreatorImpl
  • add newArray() and blockExpr() overloads that accept Class
  • always use the classic javac translation strategy of lambdas
  • fix label binding in default-less switch statements

Fixes #457

Ladicek added 5 commits August 5, 2025 15:48
Most methods that accept a `ClassDesc` also have an overload that accept
`Class`. These two methods did not, until now.
This is because certain versions of GraalVM native image cannot handle
our custom translation strategy.

This commit is temporary and is expected to be reverted in the future.
The `switch` expressions always have to have a `default` case, but
the `switch` statements do not. If they actually don't have it,
we used to bind a single `Label` object twice, which caused
an exception. This commit fixes that.
@Ladicek Ladicek added this to the 2.0.0.Beta3 milestone Aug 5, 2025
@Ladicek Ladicek added the 2.x Issue applies to Gizmo 2.x label Aug 5, 2025
@Ladicek Ladicek moved this to In Progress in WG - Gizmo 2 Aug 5, 2025
@Ladicek Ladicek merged commit a748d0a into quarkusio:main Aug 5, 2025
1 check passed
@Ladicek Ladicek deleted the improvements branch August 5, 2025 13:59
@github-project-automation github-project-automation bot moved this from In Progress to Done in WG - Gizmo 2 Aug 5, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
2.x Issue applies to Gizmo 2.x
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

IllegalArgumentException in a switch statement without the default case
1 participant