CoverityFix-CID:1350906 Checking String equality using == or != 36/20236/2
Nimish Kapoor [Sun, 8 Jul 2018 17:59:48 +0000 (22:59 +0530)]
This code compares a java.lang.String parameter for reference equality using
the == or != operators. Requiring callers to pass only String constants or
interned strings to a method is unnecessarily fragile, and rarely leads to
measurable performance gains. Consider using the equals(Object) method instead.

Change-Id: I50f8e58356a776fae747f06c9391ec84544fc604

scilab/modules/preferences/src/java/org/scilab/modules/preferences/Component/Radiobutton.java

index d51e795..0e50ad2 100644 (file)
@@ -103,7 +103,7 @@ public class Radiobutton extends SwingScilabRadioButton implements XComponent, X
      * @param text : the attribute value.
      */
     public final void text(final String text) {
-        if (text != XConfigManager.NAV) {
+        if (!text.equals(XConfigManager.NAV)) {
             setText(text);
         } else {
             setText(null);