Quoting Vittorio Giovara (2015-07-01 17:49:10) > 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]> > ---
I do not really think this is a good idea. It's used in quite many encoders, and the notion of a compression factor for lossless encoders is quite general. The right thing to do IMO is to make the behaviour consistent across encoders, not add more private options. -- Anton Khirnov _______________________________________________ libav-devel mailing list [email protected] https://lists.libav.org/mailman/listinfo/libav-devel
