[FFmpeg-devel] [PATCH] avcodec: add properties for lossless to AVCodecParameters

Derek Buitenhuis derek.buitenhuis at gmail.com
Mon May 9 15:12:27 CEST 2016


On 5/9/2016 1:48 PM, Carl Eugen Hoyos wrote:
> This should get applied asap imo.

NAK!

This patch has at least 3 people dissenting and it doesn't actually
"fix" anything except cosmetic output on the console, which is NOT
meant to be parsed not even stable.

> Arguing that the frame should be decoded a second time to 
> get the information seems like a good argument in favour 
> of this patch.

No. This is an API and ABI level change, to fix cosmetic output
of a CLI tool.

- Derek


More information about the ffmpeg-devel mailing list