From d129874cb83375e1080b85b7564a7d6bb2861f14 Mon Sep 17 00:00:00 2001 From: Karl Williamson Date: Sun, 18 Jul 2021 12:26:13 -0600 Subject: doop.c: Rmv redundant '&' instrs Casting to U8 has the same effect as ANDing with 0xFF. Remove the redundant '&' --- doop.c | 30 +++++++++++++++--------------- 1 file changed, 15 insertions(+), 15 deletions(-) (limited to 'doop.c') diff --git a/doop.c b/doop.c index fe1d44aa7a..9c4565bfaa 100644 --- a/doop.c +++ b/doop.c @@ -964,29 +964,29 @@ Perl_do_vecset(pTHX_ SV *sv) } else { if (size == 8) - s[offset ] = (U8)( lval & 0xff); + s[offset ] = (U8) (lval ); else if (size == 16) { - s[offset ] = (U8)((lval >> 8) & 0xff); - s[offset+1] = (U8)( lval & 0xff); + s[offset ] = (U8) (lval >> 8); + s[offset+1] = (U8) (lval ); } else if (size == 32) { - s[offset ] = (U8)((lval >> 24) & 0xff); - s[offset+1] = (U8)((lval >> 16) & 0xff); - s[offset+2] = (U8)((lval >> 8) & 0xff); - s[offset+3] = (U8)( lval & 0xff); + s[offset ] = (U8) (lval >> 24); + s[offset+1] = (U8) (lval >> 16); + s[offset+2] = (U8) (lval >> 8); + s[offset+3] = (U8) (lval ); } #ifdef UV_IS_QUAD else if (size == 64) { Perl_ck_warner(aTHX_ packWARN(WARN_PORTABLE), "Bit vector size > 32 non-portable"); - s[offset ] = (U8)((lval >> 56) & 0xff); - s[offset+1] = (U8)((lval >> 48) & 0xff); - s[offset+2] = (U8)((lval >> 40) & 0xff); - s[offset+3] = (U8)((lval >> 32) & 0xff); - s[offset+4] = (U8)((lval >> 24) & 0xff); - s[offset+5] = (U8)((lval >> 16) & 0xff); - s[offset+6] = (U8)((lval >> 8) & 0xff); - s[offset+7] = (U8)( lval & 0xff); + s[offset ] = (U8) (lval >> 56); + s[offset+1] = (U8) (lval >> 48); + s[offset+2] = (U8) (lval >> 40); + s[offset+3] = (U8) (lval >> 32); + s[offset+4] = (U8) (lval >> 24); + s[offset+5] = (U8) (lval >> 16); + s[offset+6] = (U8) (lval >> 8); + s[offset+7] = (U8) (lval ); } #endif } -- cgit v1.2.1