This is a live mirror of the Perl 5 development currently hosted at https://github.com/perl/perl5
refactor op.c S_bad_type_*v
authorDaniel Dragan <bulk88@hotmail.com>
Thu, 1 Jan 2015 04:09:28 +0000 (23:09 -0500)
committerFather Chrysostomos <sprout@cpan.org>
Mon, 12 Jan 2015 05:31:17 +0000 (21:31 -0800)
commited9feedd0527ff7353894db934c3ef325519b6df
tree48f3948e95145175b33639262a880f5d02fb0eb2
parente0c6a6b8c9a5470601df2e25f77f2422fb3f2ea5
refactor op.c S_bad_type_*v

-flags arg of both funcs is unused in all callers. Move the 0 to the funcs.
 flags arg is from commit ce16c625ec in 2012
-all bad_type_gv calls are right before the end of the switch, the pushing
 of 1st 3 args and call asm ops can be merged together, leaving the 1
 string constant push as the only unique op between the 7 src code
 callers of bad_type_gv, this requires reordering the args so the only
 unique one is the last/right most one, reordering can't be done to
 bad_type_pv because each following execution point after each bad_type_pv
 is different, bad_type_pv's caller/s are not a switch statement
- commit 53e06cf030 probably overlooked the 2 PL_op_desc[type] places,
  OP_DESC is a fancier superset of PL_op_desc[type], since calling
  bad_type_pv only happens during a PP syntax error, that is not
  performance critical, so replace PL_op_desc[type] with OP_DESC and
  factor out OP to description string lookup, plus custom ops are very rare
  so this shouldn't impact the error message seen by the user

VC2003 .text section of perl521.dll before 0xc9543 after 0xC9523
embed.fnc
embed.h
op.c
proto.h