On Sat, Jan 07, 2012 at 05:12:19PM -0800, Ronald S. Bultje wrote: > On Sat, Jan 7, 2012 at 1:00 PM, Diego Biurrun <[email protected]> wrote: > > On Sat, Jan 07, 2012 at 12:40:47PM -0800, Ronald S. Bultje wrote: > >> On Wed, Jan 4, 2012 at 6:14 AM, Diego Biurrun <[email protected]> wrote: > >> > --- a/libswscale/colorspace-test.c > >> > +++ b/libswscale/colorspace-test.c > >> > @@ -47,7 +47,7 @@ int main(int argc, char **argv) > >> > av_log(NULL, AV_LOG_INFO, "memory corruption test ...\n"); > >> > sws_rgb2rgb_init(); > >> > > >> > - for(funcNum=0; ; funcNum++) { > >> > + for (funcNum = 0; /* NOTHING */; funcNum++) { > >> > >> int i; // loop counter > >> > >> Or in short, the /* NOTHING */ comment smells like "this isn't really > >> needed", and we don't do that elsewhere either. The space changes are > >> OK. > > > > Not sure what you want to tell me here. > > Remove the "/* NOTHING */".
Pushed with that changed. Diego _______________________________________________ libav-devel mailing list [email protected] https://lists.libav.org/mailman/listinfo/libav-devel
