On 04/01/2019 14:15, Nicolas George wrote: > Rostislav Pehlivanov (12019-01-04): >> Hence an AVRational is appropriate as you can have fractions between 0 and >> 1, the encoder can adjust it and it'll be agnostic. > > Yes, AVRational is fine. Producing warnings for an unexpected > denominator would be a bad idea.
Agree... I don't really understand how such a requirement could even be feasibly used in an agnostic way... you'd need some function to return the quantizer offset range for any given encoder, or you risk it just failing, or having a huge load of code in any user application (filter, mpv, whatever) that has to special case every single encoder. And also a flag saying it can consume this data at all. As a side note, I'd like to point out, that this is well into bikeshed territory; Yejun sent v1 of this set a month ago and every revision has different people wanting different things, after being silent for weeks. I can imagine it's a bit frustrating. Cheers, - Derek _______________________________________________ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel
