[FFmpeg-devel] [PATCH] avcodec/mjpeg: don't override color information from demuxer

Hendrik Leppkes h.leppkes at gmail.com
Wed Dec 5 02:33:12 EET 2018


On Wed, Dec 5, 2018 at 1:14 AM Steinar H. Gunderson
<steinar+ffmpeg at gunderson.no> wrote:
>
> On Wed, Dec 05, 2018 at 12:57:43AM +0100, Hendrik Leppkes wrote:
> > These comments are not accurate. avformat does not fill these values,
> > because outside of deprecated code it does not expose such a struct to
> > the user.
>
> Hm, I was asked on #ffmpeg-devel to update it :-) But I suppose maybe it sets
> them in AVCodecParameters instead?
>

It would. And that struct is only ever filled by avformat (or a user),
and not avcodec, so there is no overlap there either.

The real problem is that its impossible to know which component to
really trust. Why does a demuxer have more authority on the color
format then a decoder? Or vice-versa?
We don't have a solution for that, and its really something the user
application currently has to solve.

- Hendrik


More information about the ffmpeg-devel mailing list