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.us...@nablet.com

Attachment: 0001-QSV-MJPEG-video-decoder-has-been-added.patch
Description: Binary data

_______________________________________________
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
http://ffmpeg.org/mailman/listinfo/ffmpeg-devel

Reply via email to