Change 31346 by [EMAIL PROTECTED] on 2007/06/07 08:51:15
Subject: [patch] fix small errors in sv.c commentary
From: Jim Cromie <[EMAIL PROTECTED]>
Date: Wed, 06 Jun 2007 23:38:53 -0600
Message-ID: <[EMAIL PROTECTED]>
Affected files ...
... //depot/perl/sv.c#1410 edit
Differences ...
==== //depot/perl/sv.c#1410 (text) ====
Index: perl/sv.c
--- perl/sv.c#1409~31333~ 2007-06-05 03:10:33.000000000 -0700
+++ perl/sv.c 2007-06-07 01:51:15.000000000 -0700
@@ -780,16 +780,16 @@
For the sv-types that have no bodies, arenas are not used, so those
PL_body_roots[sv_type] are unused, and can be overloaded. In
something of a special case, SVt_NULL is borrowed for HE arenas;
-PL_body_roots[SVt_NULL] is filled by S_more_he, but the
+PL_body_roots[HE_SVSLOT=SVt_NULL] is filled by S_more_he, but the
bodies_by_type[SVt_NULL] slot is not used, as the table is not
-available in hv.c,
+available in hv.c.
-PTEs also use arenas, but are never seen in Perl_sv_upgrade.
-Nonetheless, they get their own slot in bodies_by_type[SVt_NULL], so
-they can just use the same allocation semantics. At first, PTEs were
-also overloaded to a non-body sv-type, but this yielded hard-to-find
-malloc bugs, so was simplified by claiming a new slot. This choice
-has no consequence at this time.
+PTEs also use arenas, but are never seen in Perl_sv_upgrade. Nonetheless,
+they get their own slot in bodies_by_type[PTE_SVSLOT =SVt_IV], so they can
+just use the same allocation semantics. At first, PTEs were also
+overloaded to a non-body sv-type, but this yielded hard-to-find malloc
+bugs, so was simplified by claiming a new slot. This choice has no
+consequence at this time.
*/
@@ -869,7 +869,7 @@
FALSE, NONV, NOARENA, FIT_ARENA(0, sizeof(HE)) },
/* The bind placeholder pretends to be an RV for now.
- Also it's marked as "can't upgrade" top stop anyone using it before it's
+ Also it's marked as "can't upgrade" to stop anyone using it before it's
implemented. */
{ 0, 0, 0, SVt_BIND, TRUE, NONV, NOARENA, 0 },
End of Patch.
