On Wed, Jan 25, 2012 at 01:57:11PM +0200, Martin Storsjö wrote:
> On Wed, 18 Jan 2012, Diego Biurrun wrote:
> >--- a/libswscale/ppc/swscale_altivec.c
> >+++ b/libswscale/ppc/swscale_altivec.c
> >@@ -99,7 +99,6 @@ yuv2planeX_altivec(const int16_t *filter, int filterSize,
> > const uint8_t *dither, int offset)
> >{
> > register int i, j;
> >- {
> > DECLARE_ALIGNED(16, int, val)[dstW];
> >
> > for (i=0; i<dstW; i++)
> >@@ -142,7 +141,6 @@ yuv2planeX_altivec(const int16_t *filter, int filterSize,
> > }
> > }
> > altivec_packIntArrayToCharArray(val, dest, dstW);
> >- }
> >}
>
> You said you were going to do this in a separate patch? If you keep
> it in the same, it'd make more sense to reindent this part too at
> the same time.
In the patch
swscale: K&R formatting cosmetics for PowerPC code
I reindent the whole file, so I thought it made more sense to keep this
patch readable. I can go the extra mile and move the reindentation into
this one, but I see little point. Not that I care which way it is to be
done as long as somebody is willing to OK the patch one way or the other.
Diego
_______________________________________________
libav-devel mailing list
[email protected]
https://lists.libav.org/mailman/listinfo/libav-devel
