[Octave-patch-tracker] [patch #7649] new function narginchk

[John W. Eaton]

Follow-up Comment #1, patch #7649 (project octave):

You need to do a few more things to properly add a function to Octave.  The
file must be listed in the appropriate scripts/*/module.mk file and a tag for
the docstring should be added to some appropriate place in one of the manual's
.txi source files.

I don't think @seealso should refer to functions that we don't have, so please
remove inputParser from the list.

Doc strings should start with a statement that is like a command, so "Check
for ...", not "Checks for ..."

"hev" in the docstring should be "have"?

Why do you split the evalin call across multiple lines?

To be consistent with the rest of Octave, please use double quotes for
character strings unless there is a specific need to use single quotes.

There is no need to put your email address in the Copyright line, but if you
would like to add an "## Author: " line, after the docstring and just before
the function line, it can go there.  See other functions for an example.

Please don't put spaces after "(" or before ")".  So write


if (nargin != 2)


not


if ( nargin != 2 )


Could you please update your changeset and attach it again?

Thanks.


    _______________________________________________________

Reply to this item at:

  <http://savannah.gnu.org/patch/?7649>

_______________________________________________
  Message sent via/by Savannah
  http://savannah.gnu.org/