octave-maintainers
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: omitting unnecessary nargout checks


From: Rik
Subject: Re: omitting unnecessary nargout checks
Date: Wed, 16 Dec 2015 09:57:40 -0800

jwe,

Thanks for cset 20909:03e4ddd49396.  I had observed that there was a lot of
unnecessary work around returning the correctly sized octave_value_list
when all of the values had already been calculated, but I hadn't had the
energy to fix it.

I'm getting some new warnings during the build.  These are simply because
nargout is declared in the DEFUN, but no longer used.

libinterp/corefcn/__ichol__.cc:181:20: warning: unused parameter 'nargout'
[-Wunused-parameter]
libinterp/corefcn/__ichol__.cc:427:20: warning: unused parameter 'nargout'
[-Wunused-parameter]
libinterp/corefcn/__ilu__.cc:129:20: warning: unused parameter 'nargout'
[-Wunused-parameter]
libinterp/corefcn/__ilu__.cc:466:20: warning: unused parameter 'nargout'
[-Wunused-parameter]
libinterp/corefcn/ellipj.cc:31:20: warning: unused parameter 'nargout'
[-Wunused-parameter]
libinterp/corefcn/filter.cc:271:20: warning: unused parameter 'nargout'
[-Wunused-parameter]
libinterp/corefcn/mgorth.cc:51:20: warning: unused parameter 'nargout'
[-Wunused-parameter]
libinterp/corefcn/nproc.cc:30:20: warning: unused parameter 'nargout'
[-Wunused-parameter]
libinterp/corefcn/pr-output.cc:3406:20: warning: unused parameter 'nargout'
[-Wunused-parameter]
libinterp/corefcn/quad.cc:137:20: warning: unused parameter 'nargout'
[-Wunused-parameter]
libinterp/corefcn/quadcc.cc:1480:20: warning: unused parameter 'nargout'
[-Wunused-parameter]
libinterp/corefcn/sylvester.cc:35:20: warning: unused parameter 'nargout'
[-Wunused-parameter]

And sylvester.cc:35 is

DEFUN (sylvester, args, nargout,

--Rik




reply via email to

[Prev in Thread] Current Thread [Next in Thread]