[MPlayer-cvslog] r35628 - trunk/libaf/af_export.c

Diego Biurrun diego at biurrun.de
Tue Dec 11 13:49:17 CET 2012


On Tue, Dec 11, 2012 at 12:34:41PM +0000, Carl Eugen Hoyos wrote:
> Diego Biurrun <diego <at> biurrun.de> writes:
> 
> > > @@ -74,7 +74,6 @@ static int control(struct af_instance_s*
> > >  
> > >      // Free previous buffers
> > > -    if (s->buf[0])
> > >        free(s->buf[0]);
> > >  
> > > @@ -183,7 +182,6 @@ static void uninit( struct af_instance_s
> > >  
> > >    if(af->setup){
> > >      af_export_t* s = af->setup;
> > > -    if (s->buf[0])
> > >        free(s->buf[0]);
> > 
> > You left indentation messed up ...
> 
> Which is a good idea given how many people lately claim 
> they have written code that they did not write...
> 
> Please do not try to threaten new developers, this is 
> definitely not welcome!

Whatever you have been smoking lately, please share ...

Diego


More information about the MPlayer-cvslog mailing list