CoverityFix-CID:1350984,1350955,1350954,1350953 Questionable use of reference 21/20221/3
Nimish Kapoor [Thu, 5 Jul 2018 20:04:36 +0000 (01:04 +0530)]
Change-Id: I25463d8075c5d5c9726cf7e8ec61fd678aa94c7f

scilab/modules/graphic_objects/src/java/org/scilab/modules/graphic_objects/figure/Figure.java

index a337f37..bf291a6 100644 (file)
@@ -958,7 +958,7 @@ public class Figure extends GraphicObject implements AxesContainer {
      * @param eventHandlerEnabled the eventHandlerEnabled to set
      */
     public UpdateStatus setEventHandlerEnable(Boolean eventHandlerEnabled) {
-        if (eventHandler.getEventHandlerEnabled() == eventHandlerEnabled) {
+        if (eventHandler.getEventHandlerEnabled().equals(eventHandlerEnabled)) {
             return UpdateStatus.NoChange;
         }
         eventHandler.setEventHandlerEnabled(eventHandlerEnabled);
@@ -1350,7 +1350,7 @@ public class Figure extends GraphicObject implements AxesContainer {
         }
 
         for (int i = 0; i < value.length; i++) {
-            if (borderOptPadding[i] != value[i]) {
+            if (!borderOptPadding[i].equals(value[i])) {
                 borderOptPadding[i] = value[i];
                 status = UpdateStatus.Success;
             }
@@ -1370,7 +1370,7 @@ public class Figure extends GraphicObject implements AxesContainer {
         }
 
         for (int i = 0; i < value.length; i++) {
-            if (gridOptPadding[i] != value[i]) {
+            if (!gridOptPadding[i].equals(value[i])) {
                 gridOptPadding[i] = value[i];
                 status = UpdateStatus.Success;
             }
@@ -1390,7 +1390,7 @@ public class Figure extends GraphicObject implements AxesContainer {
         }
 
         for (int i = 0; i < value.length; i++) {
-            if (gridOptGrid[i] != value[i]) {
+            if (!gridOptGrid[i].equals(value[i])) {
                 gridOptGrid[i] = value[i];
                 status = UpdateStatus.Success;
             }