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

[Michael Niedermayer]

Hi

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 :)


> Give a finger and see the whole hand go... ;-)

be happy that its just your hand and not your arm ;)



> 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 ...

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

I am the wisest man alive, for I know one thing, and that is that I know
nothing. -- Socrates
-------------- 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/c8b84e02/attachment.pgp>