RFR 8214971 : Replace use of string.equals("") with isEmpty()

[Pavel Rappo]

> On 6 Dec 2018, at 20:27, Jim Laskey <james.laskey at oracle.com> wrote:
> 
> I assume you automated this change, might be worthwhile to automate a verification (compare the -+ pairs.) Eyeballing the patch it seems complete, one comment changed as well.
> 
> Question: Is it probably that vm optimizes .equals(“”) to .isEmpty() and there is no net win, ie., more of an esthetic clean up than for performance?
> 

String.isEmpty might read better (subject to preference). Builds and tests on
all systems should be performed. With an automatic (batch) change like this,
there's always a possibility of trying to call isEmpty on a non-String
reference.

-Pavel