rmv/de-dup static const char array "strings"
authorDaniel Dragan <bulk88@hotmail.com>
Thu, 1 Mar 2018 21:41:59 +0000 (16:41 -0500)
committerTony Cook <tony@develop-help.com>
Wed, 7 Mar 2018 03:14:15 +0000 (14:14 +1100)
commit814eedc877a5aa1dbba0047735733e9491aa94a4
tree7fb1db6b7b8481e4ab0a19d8f39477adf38d0456
parent4d169ec9e2a0eb93170198ffad69a892f91dd199
rmv/de-dup static const char array "strings"

MSVC due to a bug doesn't merge identicals between .o'es or discard these
vars and their contents.

MEM_WRAP_CHECK_2 has never been used outside of core according to cpan grep
MEM_WRAP_CHECK_2 was removed on the "have PERL_MALLOC_WRAP" branch in
commit fabdb6c0879 "pre-likely cleanup" without explination, probably bc
it was unused. But MEM_WRAP_CHECK_2 was still left on the "no
PERL_MALLOC_WRAP" branch, so remove it from the "no" side for tidyness
since it was a mistake to leave it there if it was removed from the "yes"
side of the #ifdef.

Add MEM_WRAP_CHECK_s API, letter "s" means argument is string or static.
This lets us get rid of the "%s" argument passed to Perl_croak_nocontext at
a couple call sites since we fully control the next and only argument and
its guaranteed to be a string literal. This allows merging of 2
"Out of memory during array extend" c strings by linker now.

Also change the 2 op.h messages into macros which become string literals
at their call sites instead of "read char * from a global char **" which
was going on before.

VC 2003 32b perl527.dll section size before
  .text name
   DE503 virtual size
  .rdata name
   4B621 virtual size

after
  .text name
   DE503 virtual size
  .rdata name
   4B5D1 virtual size
av.c
doop.c
handy.h
op.h
pp.c
pp_hot.c