RFR: 8252999: Cleanup: replace .equals("") with .isEmpty() within all codebase

Phil Race prr at openjdk.java.net
Fri Sep 11 00:00:15 UTC 2020


On Sun, 6 Sep 2020 13:57:19 GMT, Dmitriy Dumanskiy <github.com+1536494+doom369 at openjdk.org> wrote:

> **isEmpty** is faster + has less byte code + easier to read. Benchmarks could be found
>   [here](https://medium.com/javarevisited/micro-optimizations-in-java-string-equals-22be19fd8416).

1) This is un-necessary churn.
2) I can't even be sure I am finding the ones in my area because there's so much here
3) The ones I can find have no need of whatever performance improvement this might bring.
I think this whole PR should be withdrawn, and there may an attempt at measuring the benefits of this for the various
cases before submitting in appropriate smaller chunks. But I'll tell you now I don't see a need for the test updates
you are making.

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

Changes requested by prr (Reviewer).

PR: https://git.openjdk.java.net/jdk/pull/29



More information about the security-dev mailing list