On Thu, Apr 14, 2011 at 10:43:50PM +0200, Stefano Sabatini wrote:
> On date Thursday 2011-04-14 22:12:36 +0200, Luca Barbato wrote:
> > MMX is always built. Drop the ifdefs
> > ---
> >  libswscale/x86/rgb2rgb.c          |    3 -
> >  libswscale/x86/rgb2rgb_template.c |  300 
> > +------------------------------------
> >  2 files changed, 1 insertions(+), 302 deletions(-)
> 
> Highly uninformative message for a committ removing 300 lines of code.

Well, I think we have already improved in the log message quality
department, but I agree there sure is still a way to go.

This should likely better have been folded into one of the other
patches - hindsight is striking back ..

> Also how if the #ifdef HAVE_BIGENDIAN removal related?

x86 is little-endian, so the big-endian case has no place in the
directory for x86 code.

Diego
_______________________________________________
libav-devel mailing list
[email protected]
https://lists.libav.org/mailman/listinfo/libav-devel

Reply via email to