[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