* Bug #13579 fixed - bar displayed useless warnings. 73/15273/2
Vincent COUVERT [Thu, 25 Sep 2014 08:17:38 +0000 (10:17 +0200)]
Change-Id: Ib3e80b49dba540bc1491e6e7f84d539f3bfaede7

scilab/CHANGES_5.5.X
scilab/modules/graphics/macros/bar.sci
scilab/modules/graphics/tests/nonreg_tests/bug_13579.dia.ref [new file with mode: 0644]
scilab/modules/graphics/tests/nonreg_tests/bug_13579.tst [new file with mode: 0644]

index d4811bc..7b2aa73 100644 (file)
@@ -207,6 +207,8 @@ Scilab Bug Fixes
 
 * Bug #13554 fixed - rubberbox returned wrong values.
 
+* Bug #13579 fixed - bar displayed useless warnings.
+
 
 Xcos Bug Fixes
 ==============
index ba7e990..3c5b9d9 100644 (file)
@@ -165,11 +165,14 @@ function  bar(varargin)
     curFig = gcf();
     immediate_drawing = curFig.immediate_drawing;
 
+    wmode = warning("query");
+    warning("off"); // See bug #13579 (some bar() calling sequences will lead to a plot() warning)
     if COLORBOOL
         plot(X,Y,COLOR); // plot manages immediate_drawing property itself to avoid flickering
     else
         plot(X,Y); // plot manages immediate_drawing property itself to avoid flickering
     end
+    warning(wmode);
 
     curFig.immediate_drawing = "off";
 
diff --git a/scilab/modules/graphics/tests/nonreg_tests/bug_13579.dia.ref b/scilab/modules/graphics/tests/nonreg_tests/bug_13579.dia.ref
new file mode 100644 (file)
index 0000000..9459d46
--- /dev/null
@@ -0,0 +1,20 @@
+// =============================================================================
+// Scilab ( http://www.scilab.org/ ) - This file is part of Scilab
+// Copyright (C) 2014 - Scilab Enterprises - Vincent COUVERT
+//
+//  This file is distributed under the same license as the Scilab package.
+// =============================================================================
+// <-- TEST WITH GRAPHIC -->
+// <-- Non-regression test for bug 13579 -->
+//
+// <-- Bugzilla URL -->
+// http://bugzilla.scilab.org/13579
+//
+// <-- Short Description -->
+// Annoying warnings when using bar(1, [1,2,3])
+warning("on");
+bar(1, [1 2 3]);
+assert_checkequal(warning("query"), "on");
+warning("off");
+bar(1, [1 2 3]);
+assert_checkequal(warning("query"), "off");
diff --git a/scilab/modules/graphics/tests/nonreg_tests/bug_13579.tst b/scilab/modules/graphics/tests/nonreg_tests/bug_13579.tst
new file mode 100644 (file)
index 0000000..6c7cd85
--- /dev/null
@@ -0,0 +1,24 @@
+// =============================================================================
+// Scilab ( http://www.scilab.org/ ) - This file is part of Scilab
+// Copyright (C) 2014 - Scilab Enterprises - Vincent COUVERT
+//
+//  This file is distributed under the same license as the Scilab package.
+// =============================================================================
+
+// <-- TEST WITH GRAPHIC -->
+
+// <-- Non-regression test for bug 13579 -->
+//
+// <-- Bugzilla URL -->
+// http://bugzilla.scilab.org/13579
+//
+// <-- Short Description -->
+// Annoying warnings when using bar(1, [1,2,3])
+
+warning("on");
+bar(1, [1 2 3]);
+assert_checkequal(warning("query"), "on");
+
+warning("off");
+bar(1, [1 2 3]);
+assert_checkequal(warning("query"), "off");