* Bug 15853 fixed: mlist('cblock') endless 'operation+:..' warnings 93/20593/3
Samuel GOUGEON [Tue, 30 Oct 2018 15:38:39 +0000 (16:38 +0100)]
  http://bugzilla.scilab.org/15853

Change-Id: I22ad073ac12def47e662dad1db2796a0497ec73f

scilab/CHANGES.md
scilab/modules/data_structures/tests/nonreg_tests/bug_15853.dia.ref [new file with mode: 0644]
scilab/modules/data_structures/tests/nonreg_tests/bug_15853.tst [new file with mode: 0644]
scilab/modules/overloading/macros/%cblock_p.sci

index 7dd0a05..0c0cf16 100644 (file)
@@ -666,6 +666,7 @@ Known issues
 * [#15827](http://bugzilla.scilab.org/show_bug.cgi?id=15828): After `bode(..'rad')`, abscissae were titled 'Fréquence' instead of 'Pulsation' in french, and datatips still shew 'Hz' instead of 'rad/s'.
 * [#15847](http://bugzilla.scilab.org/show_bug.cgi?id=15847): Scilab crashed at startup.
 * [#15850](http://bugzilla.scilab.org/show_bug.cgi?id=15850): Adding a space after a string increases its size.
+* [#15853](http://bugzilla.scilab.org/show_bug.cgi?id=15853): `mlist("cblock")` displayed endlessly "operation +: Warning adding..."
 * [#15854](http://bugzilla.scilab.org/show_bug.cgi?id=15854): `horner(1:$,4)` yielded an error. Extractions from `cblock` tables were impacted.
 * [#15878](http://bugzilla.scilab.org/show_bug.cgi?id=15878): `sgrid` and `evans` were broken.
 * [#15880](http://bugzilla.scilab.org/show_bug.cgi?id=15880): `sgrid` needed some improvements: Labeling was sometimes ambiguous ; large circles were not labeled ; data_bounds did not always take the input wn into account ; named and #RRGGBB colors specifications could not be used. `evans` needed some improvements: the block of legends hid data ; asymptotes were too visible.
diff --git a/scilab/modules/data_structures/tests/nonreg_tests/bug_15853.dia.ref b/scilab/modules/data_structures/tests/nonreg_tests/bug_15853.dia.ref
new file mode 100644 (file)
index 0000000..689ca32
--- /dev/null
@@ -0,0 +1,18 @@
+// =============================================================================
+// Scilab ( http://www.scilab.org/ ) - This file is part of Scilab
+// Copyright (C) 2018 - Samuel GOUGEON
+//
+//  This file is distributed under the same license as the Scilab package.
+// =============================================================================
+// <-- CLI SHELL MODE -->
+// <-- ENGLISH IMPOSED -->
+// <-- Non-regression test for bug 15853 -->
+//
+// <-- Bugzilla URL -->
+// http://bugzilla.scilab.org/15853
+//
+// <-- Short Description -->
+// mlist('cblock') entered an infinite "operation +: Warning adding a empty.." loop
+mlist("cblock")
+ ans  =
+  (empty table)
diff --git a/scilab/modules/data_structures/tests/nonreg_tests/bug_15853.tst b/scilab/modules/data_structures/tests/nonreg_tests/bug_15853.tst
new file mode 100644 (file)
index 0000000..dbe55b9
--- /dev/null
@@ -0,0 +1,19 @@
+// =============================================================================
+// Scilab ( http://www.scilab.org/ ) - This file is part of Scilab
+// Copyright (C) 2018 - Samuel GOUGEON
+//
+//  This file is distributed under the same license as the Scilab package.
+// =============================================================================
+
+// <-- CLI SHELL MODE -->
+// <-- ENGLISH IMPOSED -->
+
+// <-- Non-regression test for bug 15853 -->
+//
+// <-- Bugzilla URL -->
+// http://bugzilla.scilab.org/15853
+//
+// <-- Short Description -->
+// mlist('cblock') entered an infinite "operation +: Warning adding a empty.." loop
+
+mlist("cblock")
index cf6692c..7982394 100644 (file)
 
 function %cblock_p(m)
     n=size(definedfields(m),"*")
+    if n==1 then
+        mprintf(_("  (empty table)\n"))
+        return
+    end
     txt=[];
     for k=2:n
         txt=[txt string(getfield(k,m))];