Skip to content

Commit

Permalink
doop.c: Rmv redundant '&' instrs
Browse files Browse the repository at this point in the history
Casting to U8 has the same effect as ANDing with 0xFF.  Remove the
redundant '&'
  • Loading branch information
khwilliamson committed Jul 24, 2021
1 parent 320fd64 commit 6128772
Showing 1 changed file with 15 additions and 15 deletions.
30 changes: 15 additions & 15 deletions doop.c
Expand Up @@ -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
}
Expand Down

0 comments on commit 6128772

Please sign in to comment.