[Ffmpeg-devel] [PATCH] compile with demux only
Måns Rullgård
mru
Sat Nov 11 17:30:40 CET 2006
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.
--
M?ns Rullg?rd
mru at inprovide.com
More information about the ffmpeg-devel
mailing list