[FFmpeg-devel] [PATCH]Rename FF_CODEC_PROPERTY_* as AV_CODEC_PROPERTY_*

[wm4]

On Sat, 18 Jul 2015 13:58:20 +0200
Michael Niedermayer <michael at niedermayer.cc> wrote:

> On Fri, Jul 17, 2015 at 12:27:40AM +0200, Carl Eugen Hoyos wrote:
> > Hi!
> > 
> > I am not really happy with attached patch as most defines in avcodec.h 
> > use the FF_ prefix.
> > 
> > Please comment, Carl Eugen
> 
> >  doc/APIchanges         |    4 ++++
> >  libavcodec/avcodec.h   |    4 ++--
> >  libavcodec/h264.c      |    2 +-
> >  libavcodec/mjpegdec.c  |    4 ++--
> >  libavcodec/mpeg12dec.c |    2 +-
> >  libavcodec/utils.c     |    4 ++--
> >  libavcodec/version.h   |    2 +-
> >  libavcodec/vp9.c       |    2 +-
> >  libavcodec/webp.c      |    2 +-
> >  9 files changed, 15 insertions(+), 11 deletions(-)
> > 74154d7ebedba679bea09d9c150d1e605f363828  patchavcodecproperties.diff
> > diff --git a/doc/APIchanges b/doc/APIchanges
> 
> the patch is techincally ok, i dont know which prefix people prefer
> though

FF_ is just a historic mistake (and was probably kept because they're
not linker-level symbols?) - no need to repeat this mistake with new
identifiers. I expect some day all public FF_ constants will be changed
to AV_. (Maybe Libav will do this.)