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

Reimar Döffinger Reimar.Doeffinger at stud.uni-karlsruhe.de
Sat Jan 6 15:48:02 CET 2007


Hello,
On Fri, Jan 05, 2007 at 11:43:47PM +0100, Michael Niedermayer wrote:
> On Fri, Jan 05, 2007 at 04:55:16PM +0100, Reimar Döffinger wrote:
> [...]
> > > that brings us to the point that you have to fix ae_lavc.c too
> > 
> > "have"? 
> 
> yes, its highly unlikely that you would fix just a subset of the
> files which you know are affected by the same bug :)

Damn, now everyone knows how to trick me into fixing bugs. I have to be
more careful ;-)

> > I have a new patch that fixes both muxer_lavf and ae_lavc. But I have to
> > say that I really do not dynamic lib handling.
> > Since MPlayer checkout always includes the libavf dirs I am tempted to
> > just always #include "libavformat/riff.h", even if this increases the
> > risks of incompatibilities even further...
> > What do you think?
> 
> i think its ok, and btw, i think you can just commit such simple fixes to
> files i maintain without having to post patches before, i trust you to
> not break anything, at least not more then i do ...

Thanks, applied the patch. Sending a patch also means waiting a bit and
think before applying which I find useful regardless though ;-)
So assuming I understood you correctly I will replace the current fix with
#include "libavformat/riff.h" unconditionally.

Greetings,
Reimar Döffinger



More information about the MPlayer-dev-eng mailing list