On Tue, Dec 06, 2016 at 12:22:42PM +0100, Diego Biurrun wrote:
> On Tue, Dec 06, 2016 at 11:54:20AM +0100, Diego Biurrun wrote:
> > On Tue, Dec 06, 2016 at 08:43:05AM +0100, Diego Biurrun wrote:
> > > So before you push this, it has to survive Oracle.
> > 
> > Also, have you compared warnings before and after?
> > This might leave behind some unused variables and similar.
> 
> Set looks good to me overall; I haven't grepped for any leftovers though
> and did not doublecheck for warnings.

Also, make sure you drop version.h #includes that become unnecessary
after your patches. I already found some in xvmc[1].

Diego

[1] A sad day indeed for your karma. Are you certain your soul is
still immortal? ;-p
_______________________________________________
libav-devel mailing list
libav-devel@libav.org
https://lists.libav.org/mailman/listinfo/libav-devel

Reply via email to