[FFmpeg-devel] [PATCH] Modifies doxy comments for strict_std_compliance in avcodec.h to improve grammar and readability

Rodney Baker rodney.baker
Sat Jun 5 14:01:30 CEST 2010


On Sat, 5 Jun 2010 12:36:00 Michael Niedermayer wrote:
> On Wed, Jun 02, 2010 at 09:51:24PM +0930, Rodney Baker wrote:
> > $subj
> > 
> > This is my first patch submission - please be gentle :-).
> > (I did the diff with kdesvn so I hope that it is in the right format - I
> > haven't figured out how to tell it to use diff -up so I hope it does it
> > by default).
> > 
> >  avcodec.h |   10 +++++-----
> >  1 file changed, 5 insertions(+), 5 deletions(-)
> > 
> > 159f10691c188a107816a3b89744027d60451d4d  avcodec-h-20100602-1.diff
> > Index: libavcodec/avcodec.h
> > ===================================================================
> > --- libavcodec/avcodec.h	(revision 23424)
> > +++ libavcodec/avcodec.h	(working copy)
> > @@ -1347,11 +1347,11 @@
> > 
> >       * - encoding: Set by user.
> >       * - decoding: Set by user.
> >       * Setting this to STRICT or higher means the encoder and decoder
> >       will
> > 
> > -     * generally do stupid things. While setting it to inofficial or
> > lower +     * generally do stupid things, whereas setting it to
> > inofficial or lower -     * will mean the encoder might use things that
> > are not supported by all +     * will mean the encoder might produce
> > output that is not supported by all
> > 
> > -     * spec compliant decoders. Decoders make no difference between
> > normal, +     * strictly spec-compliant decoders. Decoders don't
> > differentiate between normal,
> 
> i think adding "strictly" changes the intended meaning
> 
> [...]

OK. Modified patch attached removing the word "strictly". 
-- 
===================================================
Rodney Baker VK5ZTV
rodney.baker at iinet.net.au
=================================================== 

-------------- next part --------------
A non-text attachment was scrubbed...
Name: avcodec-h-20100605-1.diff
Type: text/x-patch
Size: 1683 bytes
Desc: not available
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20100605/4d81c609/attachment.bin>



More information about the ffmpeg-devel mailing list