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

[Ivan Uskov]

Hello Michael,

Monday, August 3, 2015, 12:14:39 PM, you wrote:

MN> 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.

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

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


MN> [...]


Thank, please look the attached updated.

-- 
Best regards,
 Ivan                            mailto:ivan.uskov at nablet.com
-------------- next part --------------
A non-text attachment was scrubbed...
Name: 0001-QSV-MJPEG-video-decoder-has-been-added.patch
Type: application/octet-stream
Size: 7979 bytes
Desc: not available
URL: <http://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20150803/566beef3/attachment.obj>