Hi Carnë
Thanks for the tips, they all sound like a good idea - I'll aim to follow them
for future code. (Yup, I've been trying to keep the code so that it also works
in Matlab where possible.)
When you're ready with everything else just go ahead and release the package, I
can't guarantee I'll be able to tidy up the remaining function quickly so I
don't want to hold things up.
This email address is fine, thanks for adding it.
Thanks,
Adam
> From: carandraug+...@gmail.com
> Date: Tue, 25 Sep 2012 14:51:42 +0200
> Subject: Re: [OctDev] Analyze75
> To: adamaitkenh...@hotmail.com
> CC: octave-dev@lists.sourceforge.net
>
> 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
