* Bug 14902 fixed: now array(k)=[] works for handles 70/20670/3
St├ęphane MOTTELET [Thu, 20 Dec 2018 19:42:08 +0000 (20:42 +0100)]
http://bugzilla.scilab.org/show_bug.cgi?id=14902

Change-Id: I907d1640bf1bc2469a958fe3d4714c2fcc33dbcd

scilab/CHANGES.md
scilab/modules/ast/src/cpp/ast/visitor_common.cpp
scilab/modules/ast/tests/nonreg_tests/bug_14902.tst [new file with mode: 0644]

index d60bf61..7c3880a 100644 (file)
@@ -512,6 +512,7 @@ Known issues
 * [#14840](http://bugzilla.scilab.org/show_bug.cgi?id=14840): `load()` is unable to load old data files saved with Scilab < 5.4.0 out of the SOD format.
 * [#14854](http://bugzilla.scilab.org/show_bug.cgi?id=14854): The `error_table` page needed to be updated.
 * [#14897](http://bugzilla.scilab.org/show_bug.cgi?id=14897): `xinfo` is poor and useless. It is declared obsolete.
+* [#14902](http://bugzilla.scilab.org/show_bug.cgi?id=14902): `array[k] = []` dit not work for handles (Regression)
 * [#14906](http://bugzilla.scilab.org/show_bug.cgi?id=14906): The `histplot` normalization was broken.
 * [#14915](http://bugzilla.scilab.org/show_bug.cgi?id=14915): `ascii(10)` makes `mprintf()` stop printing (REGRESSION).
 * [#14988](http://bugzilla.scilab.org/show_bug.cgi?id=14988): The example of Xcos diagram on the DERIV help page was distorted.
index d8032d1..8416e71 100644 (file)
@@ -1837,7 +1837,7 @@ types::InternalType* insertionCall(const ast::Exp& e, types::typed_list* _pArgs,
             }
             else
             {
-                pOut = pH->insert(_pArgs, _pInsert);
+                pOut = pH->remove(_pArgs);
             }
         }
         else if (_pVar->isStruct())
diff --git a/scilab/modules/ast/tests/nonreg_tests/bug_14902.tst b/scilab/modules/ast/tests/nonreg_tests/bug_14902.tst
new file mode 100644 (file)
index 0000000..596965c
--- /dev/null
@@ -0,0 +1,21 @@
+// =============================================================================
+// Scilab ( http://www.scilab.org/ ) - This file is part of Scilab
+// Copyright (C) 2018 - St├ęphane MOTTELET
+//
+//  This file is distributed under the same license as the Scilab package.
+// =============================================================================
+//
+// <-- NO CHECK REF -->
+// <-- TEST WITH GRAPHICS -->
+//
+// <-- Non-regression test for bug 14904 -->
+//
+// <-- Bugzilla URL -->
+// http://bugzilla.scilab.org/14902
+//
+// <-- Short Description -->
+// "array[k] = []" does not work for handles (Regression)
+
+h=[gca();gcf()];
+h(2)=[];
+assert_checkequal(size(h),[1,1]);
\ No newline at end of file