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