[MPlayer-dev-eng] [PATCH] some muxer_lavf mess cleanup

Michael Niedermayer michaelni at gmx.at
Fri Jan 5 16:22:24 CET 2007


Hi

On Fri, Jan 05, 2007 at 11:31:33AM +0100, Reimar Döffinger wrote:
> Hello,
> attached patch does:
> 1) include the proper header (riff.h) instead of adding our own function
> prototypes for all those functions

ok of course


> 2) add missing prototype for codec_get_id and CodecTag typedef

ok of course


> 3) remove useless extern for function prototypes

ok of course


> 4) mp_wav_tags is of type CodecTag[] not int[]!!! Lazyness is not an
> excuse for that kind of hacks

oops, thats a little embarassing but it was not exactly lazyness but rather
limitless stupidity ... when the compiler didnt like CodecTag i thought it was
just a typdef int with fourcc in it (of course thats not possible and i should
have remembered my own code ...) and as its used for example in 
libmpcodecs/ae_lavc.c which shouldnt depend on headers from lavf i thought
cheating and using int would do ...

that brings us to the point that you have to fix ae_lavc.c too


> 5) sizeof() is of type long, thus %d is the wrong printf format. Though
> I suggest fixing by setting first ctx->extradata_size, thus also
> avoiding some code duplication.

ok


[...]
-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

The educated differ from the uneducated as much as the living from the
dead. -- Aristotle 
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 189 bytes
Desc: not available
URL: <http://lists.mplayerhq.hu/pipermail/mplayer-dev-eng/attachments/20070105/bafa89fd/attachment.pgp>


More information about the MPlayer-dev-eng mailing list