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

Michael Niedermayer michael at niedermayer.cc
Mon Aug 3 11:14:39 CEST 2015


On Mon, Aug 03, 2015 at 11:36:09AM +0300, Ivan Uskov 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.

its quite possible that other decooders like hevc need such checks
too

btw, some examples of header dependancies can be seen with:
git grep 'deps.*_h'


[...]

-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

Frequently ignored answer#1 FFmpeg bugs should be sent to our bugtracker. User
questions about the command line tools should be sent to the ffmpeg-user ML.
And questions about how to use libav* should be sent to the libav-user ML.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 181 bytes
Desc: Digital signature
URL: <http://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20150803/3ab075b0/attachment.sig>


More information about the ffmpeg-devel mailing list