[Ffmpeg-devel] [PATCH] FLV common header.
Tue Dec 5 00:46:17 CET 2006
On Dec 3, 2006, at 12:44 AM, Allan Hsu wrote:
> On Dec 2, 2006, at 3:13 AM, Michael Niedermayer wrote:
>> 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
>>> 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.
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 Hsu <allan at counterpop dot net>
1E64 E20F 34D9 CBA7 1300 1457 AC37 CBBB 0E92 C779
More information about the ffmpeg-devel