On Fri, 21 Mar 2025 14:24:19 GMT, Mikhail Yankelevich 
<myankelev...@openjdk.org> wrote:

>> Brent Christian has updated the pull request incrementally with one 
>> additional commit since the last revision:
>> 
>>   'return' not needed in lambda
>
> test/jdk/java/lang/ref/FinalizerHistogramTest.java line 72:
> 
>> 70:             // GC and wait for at least 2 MyObjects to be ready for 
>> finalization,
>> 71:             // and one MyObject to be stuck in finalize().
>> 72:             ForceGC.wait(() -> { return ref1.refersTo(null) &&
> 
> Nitpick: I think you don't need a { return ....} here, it can be a pure 
> lambda afaik. But if you prefer this way, I don't mind in the slightest 😃

Correct 🙂

-------------

PR Review Comment: https://git.openjdk.org/jdk/pull/24143#discussion_r2008154439

Reply via email to