On 01/07/15 17:49, Vittorio Giovara wrote: > The reason for removing it from the global context ist that it is an > encoding-only parameter, and not all encoders use it. > Moreover it introudces unpredicted behaviour since some encoders fail, > some warn and some just clip on an invalid value. > > Replace it with a codec private option, and always set it to the current > default, with appropriate bounds. During the deprecation period, the > global context field will be read if the codec context field is set to > the default value. > > Signed-off-by: Vittorio Giovara <[email protected]> > --- > libavcodec/alacenc.c | 29 +++++++++++++++++++++++++++-- > libavcodec/avcodec.h | 7 ++++--- > libavcodec/flacenc.c | 14 +++++++++++--- > libavcodec/libmp3lame.c | 16 ++++++++++++---- > libavcodec/libopenjpegenc.c | 11 ++++++++++- > libavcodec/libopusenc.c | 14 +++++++++++--- > libavcodec/libspeexenc.c | 26 +++++++++++++++++++------- > libavcodec/libwavpackenc.c | 41 ++++++++++++++++++++++++++++++++--------- > libavcodec/libwebpenc.c | 24 +++++++++++++++++------- > libavcodec/options_table.h | 2 ++ > libavcodec/pngenc.c | 37 ++++++++++++++++++++++++++++++++----- > libavcodec/version.h | 3 +++ > libavcodec/zmbvenc.c | 32 ++++++++++++++++++++++++++++---- > 13 files changed, 208 insertions(+), 48 deletions(-) >
Seems fine. _______________________________________________ libav-devel mailing list [email protected] https://lists.libav.org/mailman/listinfo/libav-devel
