[FFmpeg-devel] [PATCH] WMA Voice decoder

Michael Niedermayer michaelni
Thu Jan 21 13:09:14 CET 2010


On Thu, Jan 21, 2010 at 11:43:00AM +0000, M?ns Rullg?rd wrote:
> Stefano Sabatini <stefano.sabatini-lala at poste.it> writes:
> 
> > On date Wednesday 2010-01-20 16:56:25 -0500, Ronald S. Bultje encoded:
> > [...]
> >> +/**
> >> + * Initialize decoder.
> >> + */
> > [...]
> >
> > Just a note on the doxy, please use third person,
> 
> Please don't.  It looks weird to me, and I've never seen any other

so does you pink bike look to me ;)


> documentation do it.  It's time we stopped this madness.

java*

besides if we are already nitpicking on irrelevant docu style
ronald mixes descriptions of what a function does jn 3rd & 2nd
person and there are also descriptions of what a funtion is
compared to what it does. (and given my crappy engrish i suspect
there are more forms or different forms as well ...)
No complaint, i ve no problem with it
and rather see it as a mild waste of time to change instead of
working on the code, i just thought i mention


[...]

-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

Asymptotically faster algorithms should always be preferred if you have
asymptotical amounts of data
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 189 bytes
Desc: Digital signature
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20100121/8c6290e7/attachment.pgp>



More information about the ffmpeg-devel mailing list