On 25 September 2012 11:17, adam aitkenhead <adamaitkenh...@hotmail.com> wrote:
> Hi Carnë,
>
> Attached are those m files for analyze75info and analyze75read - I've
> reformatted them to use Matlab's syntax.  I've still to tidy up
> analyze75write, but I'll send it on later this week if I can.
>
> Adam

Hi Adam

Thank you for the functions. I have just made the commit and added
them to the package. I have added your e-mail to the copyright notice
but let me know if you have another address that you prefer. I can
wait a bit more to have the whole analyze75 set of functions.

If you are already tidying up the next function, would be cool if you
could follow some of the standards of Octave core (most in Octave
Forge don't follow it but I find they good standards). Here's some:

  * indent the function body (this will save from writing end %function)
  * spaces after function names
  * no spaces after variables for indexing
  * user @var for variables name in the help text (do not capitalize them)

I'd suggest the others but it seems to me that you want to keep your
code Matlab compatible. Is that correct?

Carnë

------------------------------------------------------------------------------
Live Security Virtual Conference
Exclusive live event will cover all the ways today's security and 
threat landscape has changed and how IT managers can respond. Discussions 
will include endpoint security, mobile security and the latest in malware 
threats. http://www.accelacomm.com/jaw/sfrnl04242012/114/50122263/
_______________________________________________
Octave-dev mailing list
Octave-dev@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/octave-dev

Reply via email to