On Tue, Jul 12, 2016 at 04:21:01PM +0200, Alexandra Hájková wrote:
> > IMO more compact:
> 
> The code is nicely aligned now, I don't want it to be more compact

So keep it.

> >> --- a/libavcodec/x86/hevcdsp_init.c
> >> +++ b/libavcodec/x86/hevcdsp_init.c
> >> @@ -45,6 +45,39 @@ LFC_FUNCS(uint8_t, 10)
> >> +idct_dc_proto(4, 8,mmxext);
> >> +idct_dc_proto(8, 8,mmxext);
> >> +idct_dc_proto(16,8,  sse2);
> >> +idct_dc_proto(32,8,  sse2);
> >> +
> >> +idct_dc_proto(32,8,  avx2);
> >
> > spaces after ','
> 
> again, the code is nicely aligned and I'd prefer not to change it for no 
> reason

Look at how it's done in the rest of the file.

Diego
_______________________________________________
libav-devel mailing list
libav-devel@libav.org
https://lists.libav.org/mailman/listinfo/libav-devel

Reply via email to