[FFmpeg-devel] Fwd: [issue2563] AVI encoder create a file (contains multiple video and audio streams) with uncorrect fourcc tag
Michael Niedermayer
michaelni
Tue Jan 25 22:39:52 CET 2011
On Mon, Jan 24, 2011 at 09:12:58PM -0500, Ronald S. Bultje wrote:
> Hi guys,
>
> posting next patch:
>
> longstone <zhibing.min at hotmail.com> added the comment:
> > Add error check in avi_write_header() function.
>
> Looks fine to me, although I guess 99 should be 100 (100 streams means
> IDs 00-99).
>
> Ronald
> avi.h | 1 +
> avienc.c | 10 ++++++++--
> 2 files changed, 9 insertions(+), 2 deletions(-)
> b21af502b7b85ec94eecd39021ce08aee4b06e3f _Issue_2563__Resolution.patch
> Index: libavformat/avi.h
> ===================================================================
> --- libavformat/avi.h (revision 26402)
> +++ libavformat/avi.h (working copy)
> @@ -32,6 +32,7 @@
>
> #define AVI_MAX_RIFF_SIZE 0x40000000LL
> #define AVI_MASTER_INDEX_SIZE 256
> +#define AVI_MAX_STREAM_COUNT 99 //AVI format is able to assign stream ID for at most 99 streams
tabs
also we could export the max stream value in AVOutputFormat, thats not needed
to make me happy about the patch but it seems somehow nicer as it provides more
information for the outside and earlier without added complexity
actually the check could be factored out into common code
[...]
--
Michael GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB
In a rich man's house there is no place to spit but his face.
-- Diogenes of Sinope
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 198 bytes
Desc: Digital signature
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20110125/32130b70/attachment.pgp>
More information about the ffmpeg-devel
mailing list