[Ffmpeg-devel] [PATCH] FLV common header.

[Allan Hsu]

On Dec 3, 2006, at 12:44 AM, Allan Hsu wrote:

>
> On Dec 2, 2006, at 3:13 AM, Michael Niedermayer wrote:
>
>> Hi
>>
>> On Fri, Dec 01, 2006 at 08:04:54PM -0800, Allan Hsu wrote:
>>> The attached patch adds a common header, flv.h, that defines various
>>> common FLV container values between the FLV muxer and demuxer. It
>>> moves some defines from the muxer to enums in the common header and
>>> it changes the hardcoded values in both flvenc.c and flvdec.c to the
>>> new values in the common header. It should not change the  
>>> behavior of
>>> any of the code.
>>>
>>> If this patch is acceptable, my intention is to use the common  
>>> header
>>> values in a forthcoming series of onMetaData tag reading patches.
>>
>> patch ok except a few nitpicks
>
> [...]
>
> Here's the same patch with your suggestions taken into account.
[...]
> <flv_commonheader.patch>

Has anybody had a chance to look at the updated patch yet? I would  
like to know if I need to do any more work on it or if I can move on  
to writing an updated version of my metadata parsing patch.

	-Allan

--
Allan Hsu <allan at counterpop dot net>
1E64 E20F 34D9 CBA7 1300  1457 AC37 CBBB 0E92 C779