forked from enlightenment/efl
trying to clean up ccpcheck complaints - but this one is just stupid?
makes zero sense. as such so far cppcheck has only found maybe 10% of its grumbling and bitching as real bugs. shutting it up in some cases is possible. in others it's possible BUT with a possible performance penalty. that's unacceptable. wondering what to do. SVN revision: 51557
This commit is contained in:
parent
8601258d3f
commit
079d796fbe
|
@ -117,7 +117,6 @@ evas_common_blit_rectangle(const RGBA_Image *src, RGBA_Image *dst, int src_x, in
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|
||||||
|
|
||||||
/****************************************************************************/
|
/****************************************************************************/
|
||||||
|
|
||||||
static void
|
static void
|
||||||
|
@ -126,8 +125,7 @@ evas_common_copy_rev_pixels_c(DATA32 *src, DATA32 *dst, int len)
|
||||||
DATA32 *dst_end = dst + len;
|
DATA32 *dst_end = dst + len;
|
||||||
|
|
||||||
src += len - 1;
|
src += len - 1;
|
||||||
while (dst < dst_end)
|
while (dst < dst_end) *dst++ = *src--;
|
||||||
*dst++ = *src--;
|
|
||||||
}
|
}
|
||||||
|
|
||||||
|
|
||||||
|
@ -239,24 +237,24 @@ static void
|
||||||
evas_common_copy_pixels_c(DATA32 *src, DATA32 *dst, int len)
|
evas_common_copy_pixels_c(DATA32 *src, DATA32 *dst, int len)
|
||||||
{
|
{
|
||||||
DATA32 *dst_end = dst + len;
|
DATA32 *dst_end = dst + len;
|
||||||
|
|
||||||
while (dst < dst_end)
|
while (dst < dst_end) *dst++ = *src++;
|
||||||
*dst++ = *src++;
|
|
||||||
}
|
}
|
||||||
#endif
|
#endif
|
||||||
|
|
||||||
#ifdef BUILD_MMX
|
#ifdef BUILD_MMX
|
||||||
static void
|
static void
|
||||||
evas_common_copy_pixels_mmx(DATA32 *src, DATA32 *dst, int len)
|
evas_common_copy_pixels_mmx(DATA32 *src, DATA32 *dst, int len)
|
||||||
{
|
{ // XXX cppcheck: [./src/lib/engines/common/evas_blit_main.c:248]: (error) Invalid number of character ({). Can't process file.
|
||||||
|
// so... wtf? what's wrong with this { ? or anytrhing surrounding it?
|
||||||
DATA32 *dst_end, *dst_end_pre;
|
DATA32 *dst_end, *dst_end_pre;
|
||||||
#ifdef ALIGN_FIX
|
#ifdef ALIGN_FIX
|
||||||
intptr_t src_align;
|
intptr_t src_align;
|
||||||
intptr_t dst_align;
|
intptr_t dst_align;
|
||||||
|
|
||||||
src_align = (intptr_t)src & 0x3f; /* 64 byte alignment */
|
src_align = (intptr_t)src & 0x3f; /* 64 byte alignment */
|
||||||
dst_align = (intptr_t)dst & 0x3f; /* 64 byte alignment */
|
dst_align = (intptr_t)dst & 0x3f; /* 64 byte alignment */
|
||||||
|
|
||||||
if ((src_align != dst_align) ||
|
if ((src_align != dst_align) ||
|
||||||
((src_align & 0x3) != 0))
|
((src_align & 0x3) != 0))
|
||||||
{
|
{
|
||||||
|
@ -283,8 +281,7 @@ evas_common_copy_pixels_mmx(DATA32 *src, DATA32 *dst, int len)
|
||||||
src += 16;
|
src += 16;
|
||||||
dst += 16;
|
dst += 16;
|
||||||
}
|
}
|
||||||
while (dst < dst_end)
|
while (dst < dst_end) *dst++ = *src++;
|
||||||
*dst++ = *src++;
|
|
||||||
}
|
}
|
||||||
#endif
|
#endif
|
||||||
|
|
||||||
|
@ -326,8 +323,7 @@ evas_common_copy_pixels_mmx2(DATA32 *src, DATA32 *dst, int len)
|
||||||
src += 16;
|
src += 16;
|
||||||
dst += 16;
|
dst += 16;
|
||||||
}
|
}
|
||||||
while (dst < dst_end)
|
while (dst < dst_end) *dst++ = *src++;
|
||||||
*dst++ = *src++;
|
|
||||||
}
|
}
|
||||||
#endif
|
#endif
|
||||||
|
|
||||||
|
@ -506,8 +502,7 @@ evas_common_copy_pixels_rev_c(DATA32 *src, DATA32 *dst, int len)
|
||||||
dst_end = dst - 1;
|
dst_end = dst - 1;
|
||||||
dst = dst + len - 1;
|
dst = dst + len - 1;
|
||||||
|
|
||||||
while (dst > dst_end)
|
while (dst > dst_end) *dst-- = *src--;
|
||||||
*dst-- = *src--;
|
|
||||||
}
|
}
|
||||||
#endif
|
#endif
|
||||||
|
|
||||||
|
|
Loading…
Reference in New Issue