[FFmpeg-devel] [PATCH] avcodec: don't include vdpau_compat.h when vdpau is not enabled

James Almer jamrial at gmail.com
Thu Aug 4 17:01:05 EEST 2016


On 8/4/2016 4:23 AM, Carl Eugen Hoyos wrote:
> 2016-08-04 1:35 GMT+02:00 James Almer <jamrial at gmail.com>:
>> On 8/3/2016 7:59 PM, Carl Eugen Hoyos wrote:
>>> Hi!
>>>
>>> 2016-08-04 0:33 GMT+02:00 James Almer <jamrial at gmail.com>:
>>>> This removes unnecessary header dependencies.
>>>
>>> Why is this an advantage?
>>
>> h263, vc1 and mpeg decoder were pointlessly pulling the entire set
>> of h264 headers because of it.
> 
> So the advantage is that compilation gets measurably faster
> with your patch?

No, our headers are not the Boost package. i just want to reduce
dependencies so they don't get recompiled every time you modify
any of those 10+ unrelated headers or their dependencies.

> 
> Preprocessor checks instead of conditions have the disadvantage
> that less syntax checking is done in some cases.
> 
> Thank you, Carl Eugen
> _______________________________________________
> ffmpeg-devel mailing list
> ffmpeg-devel at ffmpeg.org
> http://ffmpeg.org/mailman/listinfo/ffmpeg-devel
> 



More information about the ffmpeg-devel mailing list