jainankitk commented on PR #15047:
URL: https://github.com/apache/lucene/pull/15047#issuecomment-3209033291
I see that the change has already been reverted. I backported the change
without realizing that unnamed variable is preview feature in JDK21. And then
fixed by replacing the same with dummy variable name, instead of reverting the
change itself.
Although, I am wondering why `RuntimeException unused` as method parameter
does not throwing warning, whereas `IndexOutOfBoundsException unnamedVariable`
does. I saw we are suppressing few other places using the annotation
`@SuppressWarnings("unused")`, but doesn't seem to be the case here? Is it
being suppressed some other way?
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: issues-unsubscr...@lucene.apache.org
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscr...@lucene.apache.org
For additional commands, e-mail: issues-h...@lucene.apache.org
