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

Reimar Döffinger Reimar.Doeffinger
Thu Jul 1 19:16:10 CEST 2010


On Thu, Jul 01, 2010 at 12:17:07AM -0700, Baptiste Coudurier wrote:
> >>What you do is only cosmetic changes. Please keep the code the way
> >>it is. If we need to change the prototype because a feature requires
> >>it, we will change it.
> >
> >(removed for now in below patch despite following objections)
> 
> Patch is ok, and yes the int (*read)() situation sucked.

Applied then with the one hunk re-added.
And I guess the func() feature is one of those things that
look really nice until you've had a first-hand experience
of what kind of trouble it can cause (though at least in
contrast to implicitly declared functions and types
it's at least possible to imagine some uses for it)...



More information about the ffmpeg-devel mailing list