Bug 8196 fixed: Wrong error message with negative thickness 90/13190/4
Calixte DENIZET [Thu, 21 Nov 2013 10:46:24 +0000 (11:46 +0100)]
Change-Id: If6aa4b12744befd8b824a193d1a84fc8375279fc

scilab/CHANGES_5.5.X
scilab/modules/graphics/src/c/SetProperty.c
scilab/modules/graphics/tests/nonreg_tests/bug_8196.dia.ref [new file with mode: 0644]
scilab/modules/graphics/tests/nonreg_tests/bug_8196.tst [new file with mode: 0644]

index 02c0a72..d2a57d8 100644 (file)
@@ -69,6 +69,8 @@ Scilab Bug Fixes
 
 * Bug #8133 fixed - Ticks disappeared in planar 3D view.
 
+* Bug #8196 fixed - Wrong error message with negative thickness.
+
 * Bug #8231 fixed - xrect help page did not say that clipping property was inherited.
 
 * Bug #8337 fixed - mtlb_rand now uses the "uniform" rule, whatever the random rule set is.
index 8952129..ec46278 100644 (file)
@@ -73,7 +73,7 @@ int sciSetLineWidth (int iObjUID, double linewidth)
 
     if (linewidth < 0)
     {
-        Scierror(999, _("Line width must be greater than %d.\n"), 0);
+        Scierror(999, _("Wrong value for '%s' property: Must be greater or equal to %d.\n"), "thickness", 0);
         return -1;
     }
     else
@@ -97,7 +97,7 @@ int sciSetLineStyle(int iObjUID, int linestyle)
     BOOL status = FALSE;
     if (linestyle < 0)
     {
-        Scierror(999, _("The line style must be greater than %d.\n"), 0);
+        Scierror(999, _("Wrong value for '%s' property: Must be greater or equal to %d.\n"), "line_style", 0);
         return -1;
     }
     else
@@ -118,7 +118,7 @@ int sciSetMarkSize(int iObjUID, int marksize)
 {
     if (marksize < 0)
     {
-        Scierror(999, _("The mark size must be greater or equal than %d.\n"), 0);
+        Scierror(999, _("Wrong value for '%s' property: Must be greater or equal to %d.\n"), "mark_size", 0);
         return -1;
     }
     else
@@ -308,7 +308,7 @@ int sciSetPoint(int iObjUID, double *tab, int *numrow, int *numcol)
             double* currentUpperLeftPoint = NULL;
             double upperLeftPoint[3];
             int widthIndex = 2;
-            int size = *numrow * *numcol;
+            int size = *numrow **numcol;
 
             if (size != 5 && size != 4)
             {
@@ -358,7 +358,7 @@ int sciSetPoint(int iObjUID, double *tab, int *numrow, int *numcol)
             double* currentUpperLeftPoint = NULL;
             int size = 0;
 
-            size = *numrow * *numcol;
+            size = *numrow **numcol;
 
             if ((size != 7) && (size != 6))
             {
@@ -413,7 +413,7 @@ int sciSetPoint(int iObjUID, double *tab, int *numrow, int *numcol)
             int iView = 0;
             int* piView = &iView;
 
-            if ((*numrow * *numcol != 2) && (*numrow * *numcol != 3))
+            if ((*numrow **numcol != 2) && (*numrow **numcol != 3))
             {
                 Scierror(999, _("Number of elements must be %d (%d if %s coordinate).\n"), 2, 3, "z");
                 return -1;
diff --git a/scilab/modules/graphics/tests/nonreg_tests/bug_8196.dia.ref b/scilab/modules/graphics/tests/nonreg_tests/bug_8196.dia.ref
new file mode 100644 (file)
index 0000000..a49ea01
--- /dev/null
@@ -0,0 +1,22 @@
+// =============================================================================
+// Scilab ( http://www.scilab.org/ ) - This file is part of Scilab
+// Copyright (C) 2013 - Scilab Enterprises - Vincent COUVERT
+//
+//  This file is distributed under the same license as the Scilab package.
+// =============================================================================
+// <-- TEST WITH GRAPHIC -->
+// <-- Non-regression test for bug 8196 -->
+//
+// <-- Bugzilla URL -->
+// http://bugzilla.scilab.org/show_bug.cgi?id=8196
+//
+// <-- Short Description -->
+// Some error message are still not standard.
+plot(1:10);
+e=gce();
+refMsg = msprintf(_("Wrong value for ''%s'' property: Must be greater or equal to %d.\n"), "thickness", 0);
+assert_checkerror("e;e.thickness = -1;", refMsg);
+refMsg = msprintf(_("Wrong value for ''%s'' property: Must be greater or equal to %d.\n"), "line_style", 0);
+assert_checkerror("e;e.line_style = -1;", refMsg);
+refMsg = msprintf(_("Wrong value for ''%s'' property: Must be greater or equal to %d.\n"), "mark_size", 0);
+assert_checkerror("e;e.mark_size = -1;", refMsg);
diff --git a/scilab/modules/graphics/tests/nonreg_tests/bug_8196.tst b/scilab/modules/graphics/tests/nonreg_tests/bug_8196.tst
new file mode 100644 (file)
index 0000000..9904f0f
--- /dev/null
@@ -0,0 +1,25 @@
+// =============================================================================
+// Scilab ( http://www.scilab.org/ ) - This file is part of Scilab
+// Copyright (C) 2013 - Scilab Enterprises - Vincent COUVERT
+//
+//  This file is distributed under the same license as the Scilab package.
+// =============================================================================
+
+// <-- TEST WITH GRAPHIC -->
+
+// <-- Non-regression test for bug 8196 -->
+//
+// <-- Bugzilla URL -->
+// http://bugzilla.scilab.org/show_bug.cgi?id=8196
+//
+// <-- Short Description -->
+// Some error message are still not standard.
+
+plot(1:10);
+e=gce();
+refMsg = msprintf(_("Wrong value for ''%s'' property: Must be greater or equal to %d.\n"), "thickness", 0);
+assert_checkerror("e;e.thickness = -1;", refMsg);
+refMsg = msprintf(_("Wrong value for ''%s'' property: Must be greater or equal to %d.\n"), "line_style", 0);
+assert_checkerror("e;e.line_style = -1;", refMsg);
+refMsg = msprintf(_("Wrong value for ''%s'' property: Must be greater or equal to %d.\n"), "mark_size", 0);
+assert_checkerror("e;e.mark_size = -1;", refMsg);