RFR: 8215281: Use String.isEmpty() where applicable in java.base

Claes Redestad claes.redestad at oracle.com
Thu Dec 13 13:17:05 UTC 2018


On 2018-12-13 14:06, Daniel Fuchs wrote:
> Looks good Claes!

Thanks!

> I eyeballed the rest of the patch and found
> nothing wrong - but with a patch this size
> it would be easy to miss something.

Yes, I've gone through it a couple of times now
to be sure.

> 
> Were you able to measure any improvement after
> patching?

There's a tiny reduction in bytecode executed
in initPhase1, but on any real measures any
improvement is in the noise.

/Claes

> 
> best regards,
> 
> -- daniel


More information about the core-libs-dev mailing list