You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@lucene.apache.org by "rmuir (via GitHub)" <gi...@apache.org> on 2025/02/27 17:50:21 UTC

[PR] fix lambdas for java 23 [lucene]

rmuir opened a new pull request, #14308:
URL: https://github.com/apache/lucene/pull/14308

   After the upgrade to java 23, my editor is flooded with warnings of unused variables from lambdas. Fix them.
   
   I also downloaded eclipse, installed it, checked all possible compiler options, compared the resulting file and ensured out eclipse compiler file is synced up, so that everything is explicit.


-- 
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-unsubscribe@lucene.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscribe@lucene.apache.org
For additional commands, e-mail: issues-help@lucene.apache.org


Re: [PR] fix lambdas for java 23 [lucene]

Posted by "rmuir (via GitHub)" <gi...@apache.org>.
rmuir merged PR #14308:
URL: https://github.com/apache/lucene/pull/14308


-- 
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-unsubscribe@lucene.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscribe@lucene.apache.org
For additional commands, e-mail: issues-help@lucene.apache.org


Re: [PR] fix lambdas for java 23 [lucene]

Posted by "rmuir (via GitHub)" <gi...@apache.org>.
rmuir commented on PR #14308:
URL: https://github.com/apache/lucene/pull/14308#issuecomment-2689170929

   Thanks, the change is just a bit annoying  from noise perspective. I will merge up main first, to make sure there aren't any new lambdas in recent commits that anger the check.


-- 
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-unsubscribe@lucene.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscribe@lucene.apache.org
For additional commands, e-mail: issues-help@lucene.apache.org