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

[Allan Hsu]

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.

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

-------------- next part --------------
A non-text attachment was scrubbed...
Name: flv_commonheader.patch
Type: application/octet-stream
Size: 13167 bytes
Desc: not available
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20061203/9a05d10f/attachment.obj>
-------------- next part --------------