[FFmpeg-devel] [PATCH] fix check in dyn_buf_write that gcc 4.3 optimizes away
Reimar Döffinger
Reimar.Doeffinger
Sun Oct 5 00:58:20 CEST 2008
On Sat, Oct 04, 2008 at 11:52:48PM +0200, Michael Niedermayer wrote:
> On Sat, Oct 04, 2008 at 11:28:08PM +0200, Reimar D?ffinger wrote:
> > On Sat, Oct 04, 2008 at 10:52:59PM +0200, Michael Niedermayer wrote:
> > > On Sat, Oct 04, 2008 at 10:02:35PM +0200, Reimar D?ffinger wrote:
> > > > Btw. with function also has a weird "return -1234;" that
> > > > looks a lot like nonsense to me...
> > >
> > > well, it looks like nonsense to me too :)
> >
> > Maybe someone can clean the whole file up a bit?
>
> thank you for volunteering, this is very much appreciated :))))
To my dismay it is not the only half-finished/unapplied cleanup patch I
have lying around, so if you wait for me it might take quite some
time...
> > Attached patch touches some places I found particularly weird/ugly,
> > I did not even test compilation though.
>
> well it would look ok if it where split cleanly and had been tested
I do not even know what uses that code and thus not how to really test
it...
Testing compilation and make test alone is not a big problem though
(neither is splitting once I un-mess my git tree..).
More information about the ffmpeg-devel
mailing list