Mercurial > octave

diff libinterp/corefcn/__ilu__.cc @ 20909:03e4ddd49396

Find changesets by keywords (author, files, the commit message), revision number or hash, or revset expression.
omit unnecessary nargout checks * paramdemo.cc, __ichol__.cc, __ilu__.cc, balance.cc, dassl.cc, eig.cc, ellipj.cc, filter.cc, givens.cc, hess.cc, lsode.cc, lu.cc, mgorth.cc, nproc.cc, pr-output.cc, quad.cc, quadcc.cc, sylvester.cc, urlwrite.cc, variables.cc, ccolamd.cc, colamd.cc, qr.cc: Don't call print_usage based on value of nargout. If nargout doesn't alter function behavior and returning extra values is inexpensive, simply return them instead of checking nargout.
author John W. Eaton <jwe@octave.org>
date Wed, 16 Dec 2015 11:41:11 -0500
parents c07bee629973
children a3359fe50966
line wrap: on
line diff
--- a/libinterp/corefcn/__ilu__.cc	Wed Dec 16 10:25:05 2015 +0100
+++ b/libinterp/corefcn/__ilu__.cc	Wed Dec 16 11:41:11 2015 -0500
@@ -136,7 +136,7 @@
 {
   int nargin = args.length ();
 
-  if (nargout > 2 || nargin < 1 || nargin > 2)
+  if (nargin < 1 || nargin > 2)
     print_usage ();
 
   octave_value_list retval (2);
@@ -473,7 +473,7 @@
 {
   int nargin = args.length ();
 
-  if (nargout != 2 || nargin < 1 || nargin > 3)
+  if (nargin < 1 || nargin > 3)
     print_usage ();
 
   std::string milu = "off";
@@ -940,7 +940,7 @@
 {
   int nargin = args.length ();
 
-  if (nargout < 2 || nargout > 3 || nargin < 1 || nargin > 5)
+  if (nargin < 1 || nargin > 5)
     print_usage ();
 
   octave_value_list retval;