diff options
author | David Mitchell <davem@iabyn.com> | 2018-01-08 15:42:23 +0000 |
---|---|---|
committer | David Mitchell <davem@iabyn.com> | 2018-01-19 11:24:55 +0000 |
commit | b70edc9d489a5c35fa1082bd8d3bf971c06bf762 (patch) | |
tree | 89cfe925450b795de765f3dd8f49ab5494994fad | |
parent | 82e3a3828dedea0e9162b046b45c56394740d72e (diff) | |
download | perl-b70edc9d489a5c35fa1082bd8d3bf971c06bf762.tar.gz |
S_pmtrans(): always use op_private flag variables
Various flag vars are set early on, such as:
const I32 complement = o->op_private & OPpTRANS_COMPLEMENT;
but sometimes these vars weren't being used, and op_private was being
tested again.
-rw-r--r-- | op.c | 5 |
1 files changed, 2 insertions, 3 deletions
@@ -6632,9 +6632,8 @@ S_pmtrans(pTHX_ OP *o, OP *expr, OP *repl) tbl = (short*)PerlMemShared_calloc( /* one slot for 'extra len' count and one slot * for possible storing of last replacement char */ - (o->op_private & OPpTRANS_COMPLEMENT) && - !(o->op_private & OPpTRANS_DELETE) ? 258 : 256, - sizeof(short)); + (complement && !del) ? 258 : 256, + sizeof(short)); cPVOPo->op_pv = (char*)tbl; if (complement) { |