[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 23:59:36 Rodney Baker wrote:
> 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.

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