[FFmpeg-devel] [PATCH] mxfdec: make it work with other calling conventions

Michael Niedermayer michaelni
Wed Jun 30 02:13:07 CEST 2010


On Tue, Jun 29, 2010 at 11:36:50PM +0100, M?ns Rullg?rd wrote:
> Reimar D?ffinger <Reimar.Doeffinger at gmx.de> writes:
[...]
> >> >+#define METADATA_READ_FUNC(name) int name(void *arg, ByteIOContext *pb, int tag, int size, UID uid)
> >> 
> >> I don't like the #define.
> >
> > Well, the alternative is having to change every single read function
> > manually in case you'd ever need an additional parameter.
> > And I found it very hard to find them.
> 
> I'm with Reimar on this one.  

> If you don't like it, fix it in some
> other way, but don't just say no.

so we can finally export libavutil/intreadwrite.h
you know people do alraedy use it, it is de facto a public header
or do you need more time for supporting this another way?


[...]

-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

No great genius has ever existed without some touch of madness. -- Aristotle
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 189 bytes
Desc: Digital signature
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20100630/befe2f19/attachment.pgp>



More information about the ffmpeg-devel mailing list