* Bug #12115 fixed - phaseplot(frq,db,phi) did not work. 05/9805/1
Vincent COUVERT [Tue, 27 Nov 2012 08:36:15 +0000 (09:36 +0100)]
Change-Id: I44aa04393d444b2378b63292d3a60861efba1ab0

scilab/CHANGES_5.4.X
scilab/modules/cacsd/macros/phaseplot.sci
scilab/modules/cacsd/tests/nonreg_tests/bug_12115.dia.ref [new file with mode: 0644]
scilab/modules/cacsd/tests/nonreg_tests/bug_12115.tst [new file with mode: 0644]

index 495dffe..1e1f455 100644 (file)
@@ -117,6 +117,8 @@ Bug fixes
 
 * Bug #12077 fixed - In help, attribute bgcolor was not handled in tables.
 
+* Bug #12115 fixed - phaseplot(frq,db,phi) did not work.
+
 
                     Changes between version 5.3.3 and 5.4.0
                     =======================================
index bee612f..bcf356d 100644 (file)
@@ -51,7 +51,7 @@ function []=phaseplot(varargin)
 
       [phi,d]=phasemag(varargin(2))
     case 3 then  //frq,db,phi
-      [frq,d,phi]=varargin(1:rhs-1)
+      [frq,d,phi]=varargin(1:rhs)
       if size(frq,2)<>size(d,2) then
         error(msprintf(_("%s: Incompatible input arguments #%d and #%d: Same column dimensions expected.\n"),..
                        fname,1,2))
diff --git a/scilab/modules/cacsd/tests/nonreg_tests/bug_12115.dia.ref b/scilab/modules/cacsd/tests/nonreg_tests/bug_12115.dia.ref
new file mode 100644 (file)
index 0000000..c1285d0
--- /dev/null
@@ -0,0 +1,20 @@
+// =============================================================================
+// Scilab ( http://www.scilab.org/ ) - This file is part of Scilab
+// Copyright (C) 2012 - Scilab Enterprises - Vincent COUVERT
+//
+//  This file is distributed under the same license as the Scilab package.
+// =============================================================================
+//
+// <-- Non-regression test for bug 12115 -->
+//
+// <-- Bugzilla URL -->
+// http://bugzilla.scilab.org/show_bug.cgi?id=12115
+//
+// <-- Short Description -->
+// phaseplot(frq,db,phi) did not work.
+s=poly(0,"s");
+h=syslin("c",(s^2+2*0.9*10*s+100)/(s^2+2*0.3*10.1*s+102.01));
+h1=h*syslin("c",(s^2+2*0.1*15.1*s+228.01)/(s^2+2*0.9*15*s+225));
+[frq,repf]=repfreq([h;h1],0.01,100);
+[phi,db]=phasemag(repf);
+phaseplot(frq,db,phi)
diff --git a/scilab/modules/cacsd/tests/nonreg_tests/bug_12115.tst b/scilab/modules/cacsd/tests/nonreg_tests/bug_12115.tst
new file mode 100644 (file)
index 0000000..dad1a5b
--- /dev/null
@@ -0,0 +1,21 @@
+// =============================================================================
+// Scilab ( http://www.scilab.org/ ) - This file is part of Scilab
+// Copyright (C) 2012 - Scilab Enterprises - Vincent COUVERT
+//
+//  This file is distributed under the same license as the Scilab package.
+// =============================================================================
+//
+// <-- Non-regression test for bug 12115 -->
+//
+// <-- Bugzilla URL -->
+// http://bugzilla.scilab.org/show_bug.cgi?id=12115
+//
+// <-- Short Description -->
+// phaseplot(frq,db,phi) did not work.
+
+s=poly(0,"s");
+h=syslin("c",(s^2+2*0.9*10*s+100)/(s^2+2*0.3*10.1*s+102.01));
+h1=h*syslin("c",(s^2+2*0.1*15.1*s+228.01)/(s^2+2*0.9*15*s+225));
+[frq,repf]=repfreq([h;h1],0.01,100);
+[phi,db]=phasemag(repf);
+phaseplot(frq,db,phi)