diff options
author | Father Chrysostomos <sprout@cpan.org> | 2012-04-04 22:22:53 -0700 |
---|---|---|
committer | Father Chrysostomos <sprout@cpan.org> | 2012-04-05 06:00:51 -0700 |
commit | 894ea76b1f298c94bba681c05aacda756c876245 (patch) | |
tree | 830c2b3c912e37280688ecab6f78a48fd515857e /lib/strict.pm | |
parent | 66c058c5af700a1bacbab597e94809954af40beb (diff) | |
download | perl-894ea76b1f298c94bba681c05aacda756c876245.tar.gz |
Fix VMS build broken by d1718a7cf5
This comment in perl.c:
/* Note: 20,40,80 used for NATIVE_HINTS */
(added by a0ed51b3 [Here are the long-expected Unicode/UTF-8 mod-
ifications.]), has apparently always been wrong. The values in
vms/vmsish.h end with 7 zeroes in hex, and are only shifted down to
one zero when assigned to cop->op_private in op.c:newSTATEOP. In
PL_hints they never have the values indicated in perl.h. So those
are actually free bits. It’s the high versions in vmsish.h that
are not free.
20 (actually 0x20000000) hasn’t been used for anything since commit
744a34f9085 (Urk -- undo previous removal of vmsish 'exit' change),
which change I don’t really understand. In any case, the comment was
never updated.
This comment in op.h:
/* NOTE: OP_NEXTSTATE and OP_DBSTATE (i.e. COPs) carry lower
* bits of PL_hints in op_private */
was added by d41ff1b8ad98 (introduce $^U), which was later reverted.
op_private does not carry the lower bits of PL_hints, but, rather,
certain higher bits of PL_hints, shifted to fit (the NATIVE_HINTS
cited above).
Due to misleading comments, I ended up breaking the VMS build in com-
mit d1718a7cf5, by using its bits for something else.
This commit moves the bits around a bit to avoid the clash, and modi-
fies the comments to reflect reality.
Diffstat (limited to 'lib/strict.pm')
-rw-r--r-- | lib/strict.pm | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/lib/strict.pm b/lib/strict.pm index 43b7ebf3fa..63a89dd8f4 100644 --- a/lib/strict.pm +++ b/lib/strict.pm @@ -15,9 +15,9 @@ subs => 0x00000200, vars => 0x00000400 ); my %explicit_bitmask = ( -refs => 0x20000000, -subs => 0x40000000, -vars => 0x80000000 +refs => 0x00000020, +subs => 0x00000040, +vars => 0x00000080 ); sub bits { |