[FFmpeg-devel] [PATCH] ffmpeg: copy color properties in case of -c:v copy.

Ronald S. Bultje rsbultje at gmail.com
Wed May 4 02:58:07 CEST 2016


Hi,

On Tue, May 3, 2016 at 8:29 PM, Michael Niedermayer <michael at niedermayer.cc>
wrote:

> On Tue, May 03, 2016 at 03:21:00PM -0400, Ronald S. Bultje wrote:
> > I have no idea why the first hunk uses ost->enc_ctx, because as far as
> > I understand, that is never used in case of -c:v copy, but this code
> > block is only entered if encoding_needed=0, which means stream_copy=1.
>
> when the code was added it was followed by a
> ret = avcodec_copy_context(ost->st->codec, ost->enc_ctx);
> later that was removed
>
>
> do you have a testcase for which this hunk makes a differnce?


ffmpeg -i any_webm_file -color_trc 4 -strict -2 -c:v copy out.webm

Will set the colorspace element in the muxer after, but not before, this
patch.

Ronald


More information about the ffmpeg-devel mailing list