diff options
author | Karl Williamson <khw@cpan.org> | 2021-07-18 12:26:13 -0600 |
---|---|---|
committer | Karl Williamson <khw@cpan.org> | 2021-07-30 06:11:45 -0600 |
commit | d129874cb83375e1080b85b7564a7d6bb2861f14 (patch) | |
tree | fc95f94296bb959bc4076469dffdf66adc1ecd59 /doop.c | |
parent | 9555181b32f9b30122a8ea4e779c2c9916cec9f8 (diff) | |
download | perl-d129874cb83375e1080b85b7564a7d6bb2861f14.tar.gz |
doop.c: Rmv redundant '&' instrs
Casting to U8 has the same effect as ANDing with 0xFF. Remove the
redundant '&'
Diffstat (limited to 'doop.c')
-rw-r--r-- | doop.c | 30 |
1 files changed, 15 insertions, 15 deletions
@@ -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 } |