[FFmpeg-devel] [PATCH] QSV MJPEG video decoder has been added.

[Hendrik Leppkes]

On Mon, Aug 3, 2015 at 10:36 AM, Ivan Uskov <ivan.uskov at nablet.com> wrote:
> Hello Hendrik,
>
> Monday, August 3, 2015, 12:45:36 AM, you wrote:
>
>
> HL> The decoder should depend on the header in configure directly already,
> HL> so its not built at all when the header is not available.
> In general I do not understanding why it necessary at all.
> All necessary headers currently available into the mfx_dispatch
> Now we try to account some kind of retrospective case which will never
> can appear of real life.
> By the way when Michael merged HEVC support from libva he did not add
> any checks into config although there is much more high probability
> that HEVC component is absent into mfx.
> The support of jpeg existing into libmfx for years so complex checks
> looks quite strange here.

Clearly the problem happened to Michael, so the problem might happen
to someone else.
The check is trivial, you already added a check for the header, so
just add it as a dependency to the decoder entry in configure as well.

- Hendrik