Count the size of padlist names in CVs (for v5.18 and later).

Fixing this for pre-v5.18 involves solving the more general problem of when
to "recurse" into nested structures, currently bodged with "SOME_RECURSION"
and friends. :-(

Avoid t/recurse.t failing on earlier versions on 32 bit platforms.

Subroutines used to be smaller than its hard-coded assumption.

Bump $VERSION to 0.79_53

This is 0.79_52 - update META.yml, and META.json

A more robust test for the size of subroutines in packages which import.

A test in t/recurse.t broke because Test::More::is() had been refactored.
Obviously that wasn't correct. Ultimately the problem was because the test
as-was wasn't minimally testing the thing that it was intended to test. It
should be now. Comments in the test explain the problem.

Reported as CPAN #102910.

As of v5.21.6, CvPADLIST() is only legal on pure Perl subroutines.

So move the call to padlist_size into the CvISXSUB() else block.

handle the multideref op, and to a limited extent UNOP_AUX in general.

[CPAN #102911]

Handle the new METHOP.

[CPAN #101071]

Bump $VERSION to 0.79_52

This is 0.79_51 - update META.yml, and META.json

Use formline for creating an OOK scalar to test.

As of 5.20.0, s/// no longer reliably triggers OOK, as COW is enabled.
Thanks to Niko Tyni for diagnosing the cause of the test failure.

CPAN #95493

Bump $VERSION to 0.79_51

This is 0.79_50 - update META.yml, and META.json

Update CHANGES.

reading tied scalar shrinks it

One of Devel-Size's tests, for a tied scalar visibly growing when the
underlying tie object grows, is failing on Perl 5.19.3 because an earlier
read of the scalar clears its PV buffer, causing the scalar to shrink.
Attached patch makes the test allow for such shrinkage.

wrong test for magicalness

Devel::Size looks at the SvMAGIC pointer for any SV that's SVt_PVMG or above.
This is incorrect.  It is actually necessary (and sufficient) to check the
SvMAGICAL flag.  Attached patch fixes.

Bump $VERSION to 0.79_50

This is 0.79 - update META.yml, and META.json

Bump $VERSION to 0.79

This is 0.78_52 - update META.yml, and META.json

Correct the error in the fix for cop_stashpv on v5.17.1 and later.

Remember to go back and test on the various blead versions *after* making the
fix for the problems found on the various stable versions.

Bump $VERSION to 0.78_52

This is 0.78_51 - update META.yml, and add META.json

As of v5.17.1, cop_stashpv no longer exists on threaded perls.

Instead, the cop's stash is indirected via cop_stashoff and PL_stashpad.
(See blead commit d4d03940c58a0177)

The cop_stash pointer isn't reference counted, so don't add its size.

See rt.perl.org #117941 for the obscure problems that this lack of reference
counting can cause.

Correct the year for the 0.78 release in the CHANGES file.

Bump $VERSION to 0.78_51

This is 0.78_50 - update META.yml, and add META.json

Note the release number, date and name in the CHANGES file.

An alternative test that CvOUTSIDE is being followed.

As of v5.17.2, CvOUTSIDE is NULL on cloned closures, unless they contain a
string eval. Hence the size of closures should differ, as long as Size.xs
is correctly following CvOUTSIDE.

(See blead commit a0d2bbd5c47035a4)

Update Devel-Size for the new form of pad list.

Test that we can measure that recursion increases the size of the PADLIST.

Stubbed subs no longer have CvOUTSIDE set.

t/code.t was testing whether Devel::Size was following CvOUTSIDE by relying
on the implementation detail that CvOUTSIDE is set for a subroutine that is
explicitly stubbed, and not set for one that is implicitly stubbed at runtime.

v5.17.0 changed the compile-time behaviour so that no stubbed subroutine has
CvOUTSIDE set, which breaks the assumption of this test. Hence change the test
to verify the new behaviour. However, this leaves us currently with no test
for CvOUTSIDE.

(Based on a patch by Zefram. see RT #83903 and perl's commit 8be227ab5eaa23f2)

Don't follow CvSTART() if CvROOT() is NULL.

For v5.17.2, blead switched to a slab allocator for OPs. During subroutine
compilation, the CV's slab is held in CvSTART(), to avoid extending every
PVCV by one more pointer. Once compilation is complete, the optree slab
pointer becomes accessible at an offset from CvROOT(), and CvSTART() is
set to point to the first OP. However, in some cases, PVCVs in the incomplete
state can become visible to Devel::Size, for example by following the
CvOUTSIDE() from BEGIN blocks. So we need to ignore CvSTART() if CvROOT() is
NULL. (Arguably we should always ignore CvSTART(), as the OP that it points to
should be accessible somewhere in the tree referenced by CvROOT(), but that
change can wait for a later date)

(see RT #83903 and perl's commit 8be227ab5eaa23f2)

add .gitignore

Spelling fix to docs from gregor herrmann [CPAN #78766]

Bump $VERSION to 0.78_50

This is 0.78 - update META.yml and META.json

Bump $VERSION to 0.78

This is 0.77_51 - update META.yml and META.json

Correct the code for structured exception handling on MSVC.

Bump $VERSION to 0.77_51

This is 0.77_50 - update META.yml, and add META.json

Skip tests in t/magic.t that use formline on 5.8.1 and 5.8.2

Those versions have a buggy formline that can trigger an assertion failure.

Refactor t/globs.t to avoid the side effects of a change to strict.pm

As of blead commit b50b20584a1bbc1a, Implement new 'use 5.xxx' plan,
use strict; will write to %^H. Move the eval that creates the subroutine
into a different scope, a new subroutine generate_glob(), so that its
outside pointer chain doesn't include gv_grew(). This avoids problems with
eval ops changing the side of the %^H copies at runtime.

Bump $VERSION to 0.77_50

This is 0.77 - update META.yml

Bump $VERSION to 0.77

This is 0.76_50 - update META.yml

Count the size of the "effective names" of a hash.

This code is somewhat cheeky, making direct structure accesses.

Count HvNAME(), the HV "aux" struct, and the mro_meta struct.

GvNAME() is shared from 5.10 onwards.

Use GvFILE_HEK(), if present, in preference to GvFILE()

Handle shared hash key scalars correctly.

Split out HEK size calculation into hek_size().

Add the shared HE overhead to the total size.

Add a comment to force emacs to use C mode.

Add a single (void **) cast to make the C code also valid as C++

Bump $VERSION to 0.76_50

This is 0.76 - update META.yml

Bump $VERSION to 0.76

This is 0.75_52 - update META.yml

Don't count PL_sv_placeholder in the size returned.

PL_sv_placeholder is a singleton, process wide, so shouldn't count as part of
the size.

sv_size() can return void once more.

All the code that relied on it returning the result of its call to check_new()
has been removed.

Reinstate 5.005_xx "support".

i.e. it compiles and passes tests.

Replace c*OPx macros with their expansions.

The expansions are 2 characters longer than the macros. The macros are not
present in 5.005.

Reinstate 5.6.x "support".

i.e. it compiles and passes tests.

Don't recurse into op_size() on op->pmnext.

It doesn't point to an OP owned by the current OP - it is used to form a linked
list that reset uses to find regexps to reset. Moreover, prior to 5.8.0 it can
end up pointing to freed memory, which results in much jollity.
[Bug 20010301.005, a.k.a. RT #5935, fixed in cb55de95c99e4650]

Reinstate weaken(), inadvertently removed by 8c394e1251fdfe38.

The test isn't testing as much as intended without it.

Magic vtables aren't freed when magic is freed, so don't count them.

They are static structures. Anything that assumes otherwise is buggy.

Correctly handle SvOOK scalars. 5.12 and later don't use SvIVX().

ppport.h defined NV if necessary, so no need to duplicate that.

Add -DPURIFY sizes to body_sizes[]

Use two macros to simplify the logic - MAYBE_OFFSET() where the normal version
subtracts a STRUCT_OFFSET(), and the purify version subtracts nothing, and
MAYBE_PURIFY() where there is not a simple relationship between the two sizes.

Use a table for SV body sizes. These incorporate the space saving post 5.8.x

For now, the sizes under -DPURIFY aren't correct.

In sv_size(), refactor the PV-derivatives to share as much code as possible.

As a side effect, PV"BM"s are now handled correctly on 5.10.0 and later.

In sv_size(), coalesce all calls to magic_size().

Bump $VERSION to 0.75_52

This is 0.75_51 - update META.yml

For other-than-gcc, generate calls to check_new(...)

Only use a static array of vtables on gcc. Some other compilers may be happy
with it, but VC certainly isn't. It's only trivial to determine gcc or not.

Bump $VERSION to 0.75_51

This is 0.75_50 - update META.yml

eval $ExtUtils::MakeMaker::VERSION to avoid warnings about _ not being numeric.

Frustratingly ExtUtils::MakeMaker->VERSION() returns the literal value of
$ExtUtils::MakeMaker::VERSION so it's just as "not numeric" as reading the
scalar directly.

Re-indent Makefile.PL consistently.

Simplify magic_size() by knowing that check_new() returns false for NULL.

Add the size of mg_len if mg_ptr is non-NULL.

With special cases for UTF-8 caching magic, which abuses mg_len, and for
mg_len == HEf_SVKEY

Follow mg_obj, which points to an SV.

Add tests for sizing magic.

The core's magic vtables are global constants, so aren't part of the size.

Parse perl.h to find the names of the vtables' names, and generate code to add
them all in new_state() as "seen".

Bump $VERSION to 0.75_50

This is 0.75 - update META.yml

Bump $VERSION to 0.75

This is 0.74_53 - update META.yml

Skip the aelemfast size comparison test for perls which predate the code change.

It was merged to maint-5.8 between 5.8.3 and 5.8.4, so 5.8.4 and later have it.
For 5.8.3 and earlier, the assumptions the test makes about sizes (based on op
structure) are bogus.

Skip GvFILE on ithreaded 5.8.8 and earlier, as it can point to freed memory.

Probably better to slightly under report sizes, than report something random or
SEGV.

Set the pointer alignment (in bits) via a C macro generated by the Makefile.PL

The previous approach of using sizeof(void *) is flat out wrong on a 64 bit
system. The right answer is 3. It gives 4. Which isn't a problem once the
pointer rotation code in check_new() isn't buggy, but isn't as efficient as
desired.

Fix typo error in bit-vector tracking mechanism, causing bogus "seen before"s.

On 64 bit platforms which allocate on 8 byte alignment (rather than 16), a low
pointer bit could get lost, resulting in new pointers being considered already
"seen". Joy, hilarity, failing tests and different results running under gdb.
"Found" by BinGOs' smoker, fixed on spectre.mongueurs.net. The help was
appreciated.

Bump $VERSION to 0.74_53

This is 0.74_52 - update META.yml

SvOK() && SvROK() is a tautology - SvROK() true has always implied SvOK() true.

So no need to check SvOK() first on 5.10 or earlier.

total_size() and size() can be XS ALIASes, now all the logic is in sv_size()

Move iteration over array values from total_size() to sv_size()

For now this requires some bodgery over when array values should be recursed
into. Arrays presented to total_size() should be properly recursed into.
However, historically the inner arrays of padlists were never traversed, which
means there is no code to deal with the special case rules for them. So to avoid
the perfect being the enemy of the good, defer writing that code, by skipping
the inner arrays of padlists for now.