Better fix for https://codereview.scilab.org/#/c/16760/ 66/16766/1
Vincent COUVERT [Thu, 2 Jul 2015 14:38:12 +0000 (16:38 +0200)]
Test: test_run fileio bug_4339

Change-Id: Iff4ae6fa2326f72d66c2092c3bb6206009ac0fb3

scilab/modules/graphics/src/c/getHandleProperty/get_mark_background_property.c
scilab/modules/graphics/src/c/getHandleProperty/get_mark_foreground_property.c
scilab/modules/graphics/src/c/getHandleProperty/get_mark_size_property.c

index a37f0d7..47a7bfe 100644 (file)
@@ -41,15 +41,14 @@ int get_mark_background_property(void* _pvCtx, int iObjUID)
 
     getGraphicObjectProperty(iObjUID, __GO_NUM_MARK_BACKGROUNDS__, jni_int, (void**)&piNumMarkBackgrounds);
 
-    if (piNumMarkBackgrounds == NULL)
-    {
-        Scierror(999, _("'%s' property does not exist for this handle.\n"), "mark_background");
-        return -1;
-    }
-
     if (numMarkBackgrounds == 0)
     {
         getGraphicObjectProperty(iObjUID, __GO_MARK_BACKGROUND__, jni_int, &piMarkBackground);
+        if (piMarkBackground == NULL)
+        {
+            Scierror(999, _("'%s' property does not exist for this handle.\n"), "mark_background");
+            return -1;
+        }
         return sciReturnDouble(_pvCtx, iMarkBackground);
     }
     else
index 79c42d2..9da3c03 100644 (file)
@@ -43,15 +43,14 @@ int get_mark_foreground_property(void* _pvCtx, int iObjUID)
 
     getGraphicObjectProperty(iObjUID, __GO_NUM_MARK_FOREGROUNDS__, jni_int, (void**)&piNumMarkForegrounds);
 
-    if (piNumMarkForegrounds == NULL)
-    {
-        Scierror(999, _("'%s' property does not exist for this handle.\n"), "mark_foreground");
-        return -1;
-    }
-
     if (numMarkForegrounds == 0)
     {
         getGraphicObjectProperty(iObjUID, __GO_MARK_FOREGROUND__, jni_int, &piMarkForeground);
+        if (piMarkForeground == NULL)
+        {
+            Scierror(999, _("'%s' property does not exist for this handle.\n"), "mark_foreground");
+            return -1;
+        }
         return sciReturnDouble(_pvCtx, iMarkForeground);
     }
     else
index 3dd6a05..b9ff5fc 100644 (file)
@@ -41,15 +41,14 @@ int get_mark_size_property(void* _pvCtx, int iObjUID)
 
     getGraphicObjectProperty(iObjUID, __GO_NUM_MARK_SIZES__, jni_int, &piNumMarkSizes);
 
-    if (piNumMarkSizes == NULL)
-    {
-        Scierror(999, _("'%s' property does not exist for this handle.\n"), "mark_size");
-        return -1;
-    }
-
     if (numMarkSizes == 0)
     {
         getGraphicObjectProperty(iObjUID, __GO_MARK_SIZE__, jni_int, &piMarkSize);
+        if (piMarkSize == NULL)
+        {
+            Scierror(999, _("'%s' property does not exist for this handle.\n"), "mark_size");
+            return -1;
+        }
         return sciReturnDouble(_pvCtx, iMarkSize);
     }
     else