On 1 October 2012 22:30, adam aitkenhead <adamaitkenh...@hotmail.com> wrote:
> I've just submitted analyze75write.m to SVN.
Nice, thank you. The code looks nice. I have added the function to the
INDEX and NEWS file, and to the see also of the other functions.
Some tips, you don't need to check the value returned by isfield. If
there is a field with that name, it will return true so "if (isfield
(x, name))" is enough, no need to "if (isfield (x, name) == 1).
It would be good practice to also have analyze75filename deal with the
filename part, just as it's doing for the other 2 functions. I'd guess
analyze75filename would take an extra argument ("write", "info" or
"read") for the part of checking if the file already exists. This
would also allow to make it return the right error message with:
error ('analyze75%s: error message.', analyze75function)
Carnë
------------------------------------------------------------------------------
Don't let slow site performance ruin your business. Deploy New Relic APM
Deploy New Relic app performance management and know exactly
what is happening inside your Ruby, Python, PHP, Java, and .NET app
Try New Relic at no cost today and get our sweet Data Nerd shirt too!
http://p.sf.net/sfu/newrelic-dev2dev
_______________________________________________
Octave-dev mailing list
Octave-dev@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/octave-dev
