Hi, On Sun, May 29, 2016 at 3:25 PM, Thilo Borgmann <thilo.borgm...@mail.de> wrote:
> Am 29.05.16 um 01:32 schrieb Michael Niedermayer: > > Hi > > > > It was suggested in the IRC meeting today that i start a vote to > > resolve if AVClass & AVOption should be added to AVCodecParameters > > This question needs to be awnsered before the next release because > > the ABI would be broken if its added afterwards > > the lack of any decission blocks the release which is worse than > > either decission, otherwise a vote might be silly for such technical > > question but eve a bad decission is better than no decission here > > > > > > The disadvanatges are: > > 1 more field in the struct > > a high level API that some feel is unneeded for AVCodecParameters > > it could be confusing to some that there are 2 different ways to > > access the fields > > people might use it as av_log() context when they intended to use a > > different context for it > > There are probably more please help fill the list if you know more > > > > The advanatges are: > > More consistent availability of AVOptions and AVClass in public > > structs > > Makes supporting multiple FFmpeg versions and distros easier and > > cleaner for applications. (reduces lists of #if version checks) > > Provides default/min/max values, allows basic validity checking within > > min/max > > Avoids mysterious crashes if an application uses avoption functions > > on AVCodecParameters > > Introspection > > Serialization support that may be usefull for ffserver or an application > > replacing ffserver > > > > > > An application that doesnt want to use AVOptions or AVClass can > > completey ignore them. > > > > Please state clearly if you agree to add AVClass&AVOption to > > AVCodecParameters or if you disagree about adding it or if you dont > > care either way > > I'm in favor of adding them. And you are not in the list either [1]... Ronald [1] https://ffmpeg.org/pipermail/ffmpeg-devel/2015-October/182207.html _______________________________________________ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel