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

[Diego Biurrun]

On Wed, Jun 16, 2010 at 06:40:34AM +0200, Michael Niedermayer wrote:
> On Wed, Jun 16, 2010 at 07:59:57AM +0930, Rodney Baker wrote:
> > On Wed, 16 Jun 2010 02:36:48 Michael Niedermayer wrote:
> > > On Wed, Jun 16, 2010 at 01:40:32AM +0930, Rodney Baker wrote:
> > > > $Subj.
> > > > 
> > > > --- libavcodec/avcodec.h	(revision 23615)
> > > > +++ libavcodec/avcodec.h	(working copy)
> > > > @@ -1361,7 +1361,7 @@
> > > > 
> > > > -#define FF_COMPLIANCE_INOFFICIAL   -1 ///< Allow inofficial extensions.
> > > > +#define FF_COMPLIANCE_UNOFFICIAL   -1 ///< Allow unofficial extensions.
> > > 
> > > breaks API
> > > 
> > > [...]
> > 
> > Is it OK to do this at the next major version bump, then, or not? 
> > 
> > (i.e. #if LIBAVCODEC_MAJOR_VERSION < 53 ...)
> 
> if the spelling ppl make a big fuzz of it then i dont mind
> otherwise iam leaning toward not burdening applications with an additional
>  change

I suggest changing it.  Typos are always counterintuitive and make an
API harder to use.

Diego