* Bug #12639 fixed - justify([], position) returned an error instead of "". 94/11594/3
Charlotte HECQUET [Mon, 27 May 2013 09:16:29 +0000 (11:16 +0200)]
Change-Id: Ia740d55c62e103985b8073509936968d44e20a2e

scilab/CHANGES_5.5.X
scilab/modules/string/macros/justify.sci
scilab/modules/string/tests/nonreg_tests/bug_12639.dia.ref [new file with mode: 0644]
scilab/modules/string/tests/nonreg_tests/bug_12639.tst [new file with mode: 0644]

index a1d7a63..1ec60f5 100644 (file)
@@ -304,3 +304,5 @@ Bug fixes
 
 * Bug #12634 fixed - ATOMS modules could not be installed from an archive file
                      already existing in SCI/contrib/archives in offline mode.
+
+* Bug #12639 fixed - justify([], position) returned an error instead of [].
index 954f456..e3427c2 100644 (file)
@@ -20,6 +20,10 @@ function t = justify(t, job)
         error(msprintf(gettext("%s: Wrong number of input arguments: %d expected.\n"),"justify",2))
     end
 
+    if t == [] then
+        t = [];
+        return;
+    end
     if type(t) <> 10 then
         error(msprintf(gettext("%s: Wrong type for input argument #%d: A string expected.\n"), "justify", 1));
     end
diff --git a/scilab/modules/string/tests/nonreg_tests/bug_12639.dia.ref b/scilab/modules/string/tests/nonreg_tests/bug_12639.dia.ref
new file mode 100644 (file)
index 0000000..838aefe
--- /dev/null
@@ -0,0 +1,19 @@
+// =============================================================================
+// Scilab ( http://www.scilab.org/ ) - This file is part of Scilab
+// Copyright (C) 2013 - Scilab Enterprises- Charlotte HECQUET
+//
+//  This file is distributed under the same license as the Scilab package.
+// =============================================================================
+//
+// <-- CLI SHELL MODE -->
+//
+// <-- Non-regression test for bug 12639 -->
+//
+// <-- Bugzilla URL -->
+// http://bugzilla.scilab.org/show_bug.cgi?id=12639
+//
+// <-- Short Description -->
+// justify([], position) returns an error instead of ""
+assert_checkequal(justify([], "l"), []);
+assert_checkequal(justify([], "r"), []);
+assert_checkequal(justify([], "c"), []);
diff --git a/scilab/modules/string/tests/nonreg_tests/bug_12639.tst b/scilab/modules/string/tests/nonreg_tests/bug_12639.tst
new file mode 100644 (file)
index 0000000..88737d9
--- /dev/null
@@ -0,0 +1,20 @@
+// =============================================================================
+// Scilab ( http://www.scilab.org/ ) - This file is part of Scilab
+// Copyright (C) 2013 - Scilab Enterprises- Charlotte HECQUET
+//
+//  This file is distributed under the same license as the Scilab package.
+// =============================================================================
+//
+// <-- CLI SHELL MODE -->
+//
+// <-- Non-regression test for bug 12639 -->
+//
+// <-- Bugzilla URL -->
+// http://bugzilla.scilab.org/show_bug.cgi?id=12639
+//
+// <-- Short Description -->
+// justify([], position) returns an error instead of ""
+
+assert_checkequal(justify([], "l"), []);
+assert_checkequal(justify([], "r"), []);
+assert_checkequal(justify([], "c"), []);