* Bug 7652 (+NR test): inserting list("") in a cells array coud be erroneous 56/19456/4
Samuel GOUGEON [Fri, 6 Oct 2017 21:09:57 +0000 (23:09 +0200)]
  http://bugzilla.scilab.org/7652

Change-Id: I730adbea724ff82758dcb9695f3b78a1503bb045

scilab/CHANGES.md
scilab/modules/data_structures/tests/nonreg_tests/bug_7652.tst [new file with mode: 0644]

index 1138e6f..5abcb97 100644 (file)
@@ -370,6 +370,7 @@ the [development mailing list](dev@lists.scilab.org) for a particular toolbox.
 * [#5278](http://bugzilla.scilab.org/show_bug.cgi?id=5278): obsolete `xset()` was still used in scripts, macros, tests and help pages.
 * [#5602](http://bugzilla.scilab.org/show_bug.cgi?id=5602): Assigning a component of a list in a cells array replaced the whole list.
 * [#5611](http://bugzilla.scilab.org/show_bug.cgi?id=5611): It was not possible to delete the row or column of a cells array.
+* [#7652](http://bugzilla.scilab.org/show_bug.cgi?id=7652): Inserting `list("")` in a cells array could be erroneous.
 * [#8297](http://bugzilla.scilab.org/show_bug.cgi?id=8297): `cat` slowness was exponential, crippling, and made it useless.
 * [#8842](http://bugzilla.scilab.org/show_bug.cgi?id=8842): Assigning the pointer of a builtin function to a new structure failed.
 * [#9297](http://bugzilla.scilab.org/show_bug.cgi?id=9297): Assigning a mlist to a structure's field failed.
diff --git a/scilab/modules/data_structures/tests/nonreg_tests/bug_7652.tst b/scilab/modules/data_structures/tests/nonreg_tests/bug_7652.tst
new file mode 100644 (file)
index 0000000..27f99fb
--- /dev/null
@@ -0,0 +1,32 @@
+// =============================================================================
+// Scilab ( http://www.scilab.org/ ) - This file is part of Scilab
+// Copyright (C) 2017 - Samuel GOUGEON
+//
+//  This file is distributed under the same license as the Scilab package.
+// =============================================================================
+
+// <-- CLI SHELL MODE -->
+// <-- NO CHECK REF -->
+
+// <-- Non-regression test for bug 7652 -->
+//
+// <-- Bugzilla URL -->
+// http://bugzilla.scilab.org/7652
+//
+// <-- Short Description -->
+// Inserting list("") in a cells array could be erroneous
+
+c = cell(2,2);
+c(2,2) = {list("")};
+assert_checkequal(c{2,2}, list(""));
+c(2,2) = {list([])};
+assert_checkequal(c{2,2}, list([]));
+c(2,2) = {list()};
+assert_checkequal(c{2,2}, list());
+
+c{2,2} = list("");
+assert_checkequal(c{2,2}, list(""));
+c{2,2} = list([]);
+assert_checkequal(c{2,2}, list([]));
+c{2,2} = list();
+assert_checkequal(c{2,2}, list());