On Wed, 25 Jan 2012, Diego Biurrun wrote:
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.
I see. Ok then.
// Martin
_______________________________________________
libav-devel mailing list
[email protected]
https://lists.libav.org/mailman/listinfo/libav-devel
