[Ffmpeg-devel] [PATCH] remove unused variables

Limin Wang lance.lmwang
Wed Mar 7 10:04:18 CET 2007


-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA1

Hi,

* Michael Niedermayer <michaelni at gmx.at> [2007-03-07 09:43:05 +0100]:

> Hi
> 
> On Wed, Mar 07, 2007 at 02:37:21PM +0800, Limin Wang wrote:
> > Hi,
> > 
> > Please review the patch. 
> > 
> > Thanks,
> > Limin
> 
> [...]
> >   the_end:
> > -    av_free(buf);
> > -    av_free(buf1);
> > +    return;
> >  }
> 
> the return statement is uneccesary except that the patch looks ok

Compile will fail for the_end label in the end if no return statement. 
A separate patch will remove the_end label, then return can be removed.

Thanks,
Limin
-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.4.6 (GNU/Linux)

iQEVAwUBRe6AEkztbf7dKiuoAQKv+wgAh8RcWzSnOlrl6mineg3FvJIHlTJ7oUSz
ge86osS9mOtTqHjxUbthZAftdT7da+S4pVdOUoIHK6NVPPKmX043sJXdk7JfHLTs
p9oRqZslrhkJNyE5abju56ApW73KU42ji8FpH694TQNiIifnWJlF+Vs7siidzvBb
fXGcsr0S8x+YyDj8djtADflKYDq0DfigeVZCgimXNYLlfBrD5nH7mD4P3M5xPV/O
v1M8xiwEzCPmkE7Bfb/6COS+6J2nrm9GSHRMu/D8SXF0DAtgUgBLphcO94pCrnYa
mRpSwXYRtD8Q6+bmEL6D8zqU2JybC7OfsGcrwPtnJWjAmT2jsCzGzg==
=mXC7
-----END PGP SIGNATURE-----




More information about the ffmpeg-devel mailing list