<html>
  <head>
    <meta content="text/html; charset=UTF-8" http-equiv="Content-Type">
  </head>
  <body bgcolor="#FFFFFF" text="#000000">
    I am not sure how important this clean up is but since you've done
    it ..<br>
    <br>
    replacing "".equals(foo) wth foo.isEmpty(()<br>
    isn't equivalent but the cases I saw where you did are either<br>
    foo == null || [!]foo.isEmpty()<br>
    eg :<br>
    -        if (fontName == null || "".equals(fontName)) {<br>
    +        if (fontName == null || fontName.isEmpty()) {<br>
    or<br>
    foo != null && [!]foo.isEmpty()<br>
    <br>
    eg:<br>
    -        if(text != null && !text.equals("")) {<br>
    +        if(text != null && !text.isEmpty()) {<br>
    <br>
    so these should be fine. Although for the line above since<br>
    you are touching it I'd like "if(" turned into "if ("<br>
    <br>
    Here's another one <br>
    <meta http-equiv="content-type" content="text/html; charset=UTF-8">
    <pre>-                            if(!keyword.equals("") && !value.equals("")) {
+                            if(!keyword.isEmpty() && !value.isEmpty()) {</pre>
    <br>
    With those updates,  +1<br>
    <br>
    -phil<br>
    <br>
    On 5/2/19, 5:01 AM, Sergey Bylokhov wrote:
    <blockquote
      cite="mid:199db90a-9b16-60ed-c079-aec7c8655ee4@oracle.com"
      type="cite">Hello.
      <br>
      Please review the fix for JDK 13.
      <br>
      <br>
      Bug: <a class="moz-txt-link-freetext" href="https://bugs.openjdk.java.net/browse/JDK-8223237">https://bugs.openjdk.java.net/browse/JDK-8223237</a>
      <br>
      Fix: <a class="moz-txt-link-freetext" href="http://cr.openjdk.java.net/~serb/8223237/webrev.00">http://cr.openjdk.java.net/~serb/8223237/webrev.00</a>
      <br>
      <br>
      This change is an equivalent of JDK-8214971[1] but for the
      java.desktop module.
      <br>
       - The string.equals("") replaced by the string.isEmpty() in all
      cases
      <br>
       - The "".equals(string) replaced by the string.isEmpty() when
      string is non-null
      <br>
      [1] <a class="moz-txt-link-freetext" href="https://bugs.openjdk.java.net/browse/JDK-8214971">https://bugs.openjdk.java.net/browse/JDK-8214971</a>
      <br>
      <br>
      <br>
      In one case I dropped string.equals("") as unneeded:
      <br>
<a class="moz-txt-link-freetext" href="http://cr.openjdk.java.net/~serb/8223237/webrev.00/src/java.desktop/share/classes/javax/swing/plaf/basic/BasicToolTipUI.java.udiff.html">http://cr.openjdk.java.net/~serb/8223237/webrev.00/src/java.desktop/share/classes/javax/swing/plaf/basic/BasicToolTipUI.java.udiff.html</a>
      <br>
      <br>
    </blockquote>
  </body>
</html>