RFR: 6543126: Level.known can leak memory

Peter Levart peter.levart at gmail.com
Wed Aug 17 11:35:07 UTC 2016



On 08/17/2016 01:16 PM, Daniel Fuchs wrote:
>>> Now that you have various findByXXX methods return Optional<Level>, you
>>> could make methods that use them more Optional-API-powered. For 
>>> example,
>>> findLevel could be written as a single expression:
>
> Right. But do you mind if I leave that for another day?
> I'm not too keen on multi-line code statements embedded
> in lambdas - but I feel like we did enough restructuring
> for this fix now ;-) 

It's just that the functional style is actually more readable in this 
case. But never mind. It's just style.

Regards, Peter



More information about the core-libs-dev mailing list