On Sat, 18 Jul 2015 13:58:20 +0200 Michael Niedermayer <mich...@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.) _______________________________________________ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel
