[FFmpeg-devel] [PATCH] Change all instances of "inofficial" to "unofficial" (except c/l option in lavc/options.c)

[Rodney Baker]

On Tue, 29 Jun 2010 02:53:12 Stefano Sabatini wrote:
> On date Sunday 2010-06-27 00:32:22 +0930, Rodney Baker encoded:
> > On Sun, 27 Jun 2010 00:27:09 Rodney Baker wrote:
> [...]
> 
> > Index: Changelog
> > ===================================================================
> > --- Changelog	(revision 23792)
> > +++ Changelog	(working copy)
> > @@ -15,9 +15,11 @@
> > 
> >  - libfaad2 wrapper removed
> >  - DTS-ES extension (XCh) decoding support
> >  - native VP8 decoder
> > 
> > +- added FF_COMPLIANCE_UNOFFICIAL
> > +- changed all instances of FF_COMPLIANCE_INOFFICIAL to use
> > FF_COMPLIANCE_UNOFFICIAL
> 
> These are API changes, should be placed in doc/APIchanges.

Moved.

> > +- added "unofficial" command line option and marked "inofficial" to be
> > removed at next major version bump.
> 
> I suggest:
> -strict inofficial replaced by -strict unofficial

Changed.

> 
> (note that "inofficial" is a named constant, not the name of an
> option.)

Quite right - thanks for the clarification.
> 
> Rest of the patch looks OK to me, I'll apply if Michael is fine with
> the change (I am as this is has been a source of confusion).
> 
> Regards.

Updated patch (rebased against r23885) attached. The revision number in 
doc/APIchanges may need updating once committed as I'm not sure what the final 
revision number will be. 


-- 
===================================================
Rodney Baker VK5ZTV
rodney.baker at iinet.net.au
=================================================== 

-------------- next part --------------
A non-text attachment was scrubbed...
Name: inofficial-20100629.patch
Type: text/x-patch
Size: 7417 bytes
Desc: not available
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20100629/cfc7edd9/attachment.bin>