[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