[Ffmpeg-devel] [PATCH] compile with demux only

Stefan Gehrer stefan.gehrer
Sun Nov 12 21:24:24 CET 2006


M?ns Rullg?rd wrote:
> Stefan Gehrer <stefan.gehrer at gmx.de> writes:
>
>   
>> M?ns Rullg?rd wrote:
>>     
>>> Stefan Gehrer <stefan.gehrer at gmx.de> writes:
>>>
>>>       
>>>> I found the time to do something about the cavs parsing.
>>>> As said before, I moved the parsing into cavs.c and added the
>>>> proper ifdefs. I also needed to make mpeg4video_split from
>>>> parser.c public and thus renamed it to ff_mpeg4video_split.
>>>> This should make it possible to build with any combination of
>>>> CONFIG_CAVSVIDEO_PARSER and CONFIG_CAVS_DECODER.
>>>>
>>>>         
>>> Patch seems ok at first glance, but needs to be split.  The renaming
>>> of mpeg4video_split to ff_mpeg4video_split and making it extern should
>>> be done first, then the cavs parser can be made to use it.  The rename
>>> and staticizing of cavs_flush should also be done separately.
>>>
>>>
>>>       
>> Done.
>> part1.diff (a public ff_mpeg4video_split) needs to be applied first as
>> part2.diff (move cavs parser to cavs.c) is on top of it.
>> Making cavs_flush static can be done later.
>>     
>
> Fine with me.  Apply unless someone (Michael) objects.
>
>   
Applied.




More information about the ffmpeg-devel mailing list