Mercurial > octave

diff libinterp/corefcn/input.cc @ 20819:f428cbe7576f

Find changesets by keywords (author, files, the commit message), revision number or hash, or revset expression.
eliminate unnecessary uses of nargin * __dsearchn__.cc, betainc.cc, bsxfun.cc, data.cc, debug.cc, det.cc, dot.cc, error.cc, file-io.cc, givens.cc, graphics.cc, hess.cc, hex2num.cc, input.cc, inv.cc, mgorth.cc, ordschur.cc, pr-output.cc, profiler.cc, rcond.cc, regexp.cc, sqrtm.cc, sub2ind.cc, sylvester.cc, syscalls.cc, sysdep.cc, tsearch.cc, urlwrite.cc, utils.cc, variables.cc: Don't use nargin variable unless it is used more than once.
author John W. Eaton <jwe@octave.org>
date Mon, 07 Dec 2015 13:54:01 -0500
parents 3d551b2ae928
children 1142cf6abc0d
line wrap: on
line diff
--- a/libinterp/corefcn/input.cc	Mon Dec 07 13:26:03 2015 -0500
+++ b/libinterp/corefcn/input.cc	Mon Dec 07 13:54:01 2015 -0500
@@ -709,11 +709,9 @@
 {
   octave_value_list retval;
 
-  int nargin = args.length ();
-
   int read_as_string = 0;
 
-  if (nargin == 2)
+  if (args.length () == 2)
     read_as_string++;
 
   std::string prompt = args(0).xstring_value ("input: unrecognized argument");
@@ -917,9 +915,7 @@
 {
   octave_value_list retval;
 
-  int nargin = args.length ();
-
-  if (nargin > 1)
+  if (args.length () > 1)
     print_usage ();
 
   unwind_protect frame;
@@ -973,11 +969,9 @@
 {
   octave_value_list retval;
 
-  int nargin = args.length ();
-
   string_vector argv = args.make_argv ("echo");
 
-  switch (nargin)
+  switch (args.length ())
     {
     case 0:
       {
@@ -1095,9 +1089,7 @@
 {
   octave_value retval;
 
-  int nargin = args.length ();
-
-  if (nargin != 1)
+  if (args.length () != 1)
     print_usage ();
 
   std::string hint = args(0).string_value ();