Reorder some code to make identical sections more apparent

Only enable careful self destruction on perl 5.25.3 and higher

Handle self destruction manually

A slightly modified version of the failing test code is as follows:

    use Variable::Magic qw<wizard cast>;

    my $wiz2 = wizard;
    my $wiz1 = wizard free =>
            sub { warn "recasting\n"; &cast($_[0], $wiz2); die ; };

    warn "result of eval = [" . eval {
     my $v = do { my $val = 123; \$val };
     study;
     &cast($v, $wiz1);
     warn "just at end of eval\n";
    } . "]\n";
    warn "just after eval\n";

On 5.24.0, this gives:

    just at end of eval
    recasting
    result of eval = []
    recasting
    just after eval

The mortal RV that is created to temporarily point at the scalar being
freed (the IV(123) above) to pass to the free method, isn't initially
freed, and is only freed by the FREETMPS in the nextstate following the
eval. When freed, it triggers another free of the IV(123), which although
it should now be under the influence of $wiz2 rather than $wiz1, it still
calls the 'free' anon sub (I don't understand why its still called, and I
haven't looked into it).

The TEMP not getting freed until after the statement following the eval is
the bug my blead patch was supposed to fix (which it does), but which
caused infinite recursion.

My fix avoids making the temporary mortal RV a TEMP on the tmps stack,
and instead stores a pointer to it in the vmg_svt_free_cleanup_ud struct.
This RV is then manually freed in both the normal and exception cleanup
paths.

Test that recasting happens only once

This fails starting from perl 5.24.

This is 0.60

Skip len magic test on negative indices on recent perls

Starting from perl 5.25.4, this does not cause len magic to be triggered
anymore. It has been temporarily reverted from the core, but is likely to
come back, so this should appease the smokes.

This fixes [RT #128989].

Bump copyright year

Update VPIT::XSHelpers to 18554226

Modulo the C++ bit... sigh.

The Big Boilerplate Factorization

This is 0.59

Remove a now obsolete workaround for context stack smashing

Since we switch stacks, we also switch context stacks, so this is no longer
necessary.

Execute magic callbacks into separate stacks

When magic triggered, stack corruption could occur since the magic callback
arguments were passed on the same stack as the ambient one. This issue has
long gone unnoticed because most ops first unpack their stack arguments
before processing magic, but also because this module is mostly used in
practice to capture magic outside of the usual program op structure were
the argument stack is relevant.

This was made visible with core change 6442877a (publicized in perl 5.22.0)
which caused get magic on transliterations to happen on the same C argument
list as a PUSHmortal. This resulted in an undefined behaviour which was
usually resolved by the magic being processed first (for example by gcc),
but cl.exe on win32 prefers to execute the PUSHmortal first, hence the
stack was corrupted.

This fixes RT #107294.

Improve support for custom ops

This was added to the core in commit 1830b3d9, which was publicized in perl
5.13.7. Note that our version is slightly different but simpler, since I
can't see how the op type might change during the call to OP_CLASS().

Correctly bless UTF-8 transliteration op objects

This is a port of commit 512ba29b from perl.

Represent nulled COPs as B::COP objects

This change was added to the core B.xs in commit 619dadb5, which was made
public in perl 5.21.7. It is pretty unlikely that magic will ever be called
for these nulled ops, but it is better to keep our chunk of B.xs in sync
with the core.

Add support for ops of class UNOP_AUX

This class has been added in commit 2f7c6295, which was publicly available
in perl 5.21.7.

Remove the code coverage link

It's outdated and now covered by cpancover.

Make sure asserts are only checked on DEBUGGING builds

This is 0.58

Remove the done_testing() comment in t/09-load-threads.t

The croaking stubs are actually imported only when the environment variable
PERL_TEST_LEANER_USES_TEST_MORE is set.

Simplify the timeout logic in t/09-load-threads.t

The old loop could actually never be executed more than one iteration.
A real good implementation would check whether any of the peers has exited,
but it would involve much more work.

Remove an unused static variable

Work around stray exits in t/09-load-threads.t

Work around stray exits in t/40-threads.t and t/41-clone.t

Update VPIT::TestHelpers to ba865c42

Avoid global lookups in the magic callbacks in t/18-opinfo.t

This will hopefull work around a rare test failure on MSWin32 (see
http://www.cpantesters.org/cpan/report/cbda218a-6c4a-1014-b9c5-a00a324d4f1e)

Delay destruction of reference return values till the current statement end

Test timely destruction of values returned from magic callbacks

Update VPIT::TestHelpers to 3ba261a5

This is 0.57

Rename a variable in t/09-load-threads.t

Also test cloning in t/09-load-threads.t

vmg_teardown() should also be called at the end of cloned threads

This is achieved by removing the "top interpreter" guard in vmg_teardown(),
which has been obsoleted by our new loaded refcount.

Update VPIT::TestHelpers to 6d578776

And list the dependencies for the 'capture' feature in BUILD_REQUIRES.

Make sure t/40-threads.t does not use the CLONE method

This is t/41-clone.t's job.

Update VPIT::TestHelpers to 3edc6d15

POSIX is now correctly required.

Allow disabling thread safety in Makefile.PL

Note that thread safety still cannot be forcefully enabled if the necessary
macros are not defined.

Make sure the load refcount is reset on global setup

Don't lock the VMG_LOADED mutex across global destruction

If this mutex falls back to the OP_REFCNT mutex, and that a coderef is
freed during global destruction, the mutex is taken twice and the process
hangs. As a general rule, it does not seem very safe so we just set
vmg_loaded to zero early and only run the global teardown callback if
it is zero. This ensures that this callback will only be called once.

Update VPIT::TestHelpers to f24eb57f

And port t/17-ctl.t to the new 'capture' feature. Capture::Tiny is no
longer needed for the optional external tests.

Reference-count global setup/teardown

This fixes surprises like this one : module loaded in thread 1, then in
thread 2, then thread 1 ends and now the module does not work anymore in
thread 2 (even though it should).

Update t/09-load-test.t

It now has better diagnostics in case of failure, and it tests asymetrical
thread termination.

Make sure MULTIPLICITY builds set PERL_IMPLICIT_CONTEXT

Add a missing volatile qualifier

Improve scheduling for the parallel loading test

Revamp the setup boot logic

All of this just to fix a couple of leaks!

Set PERL_DESTRUCT_LEVEL through the environment variable on debugging perls

Always honor $ENV{PERL_DESTRUCT_LEVEL} when it's set for a debugging perl

I.e. don't fall back to Perl::Destruct::Level if the level is not high
enough.

Use PERL_FORCE_TEST_THREADS to force testing global destruction

Revamp PERL_DESTRUCT_LEVEL handling

Allow passing level 0 to Variable::Magic::TestGlobalDestruction::import

Port thread tests to the new VPIT::TestHelpers interface

The new 'force threads test' environment variable is
PERL_FORCE_TEST_THREADS.

Thoroughly test module loading in threads

Update VPIT::TestHelpers to 4eeb5afc

Update the ActivePerl/gcc-3.4 hack for EUMM 7.04

Starting from this version, PERL_ARCHIVE is quoted so we can no longer set
it to the empty string. Instead, we remove all occurrences of PERL_ARCHIVE
in the linking step section of the Makefile, which is OK since we specify
the perl DLL in other variables. This new technique is still valid for
older versions of EUMM.

This is 0.56

Bump copyright year

Update VPIT::TestHelpers to 3d200396

Add lib.pm to BUILD_REQUIRES

Replace STATIC by static

Set the mg_flags inside vmg_sv_magicext() according to the vtbl entries

Factor the sv_magicext() calls into one vmg_sv_magicext()

This new function effectively steals the given object.

Add a couple of missing SvREFCNT_dec() after sv_magicext()

Simplify vmg_propagate_errsv_free() so that ERRSV is not used as a lvalue

Simplify vmg_pp_propagate_errsv() so that ERRSV is not used as a lvalue

Simplify vmg_call_sv() so that ERRSV is no longer used as a lvalue

It's less hackish and more resilient to core changes.

Code cleanups in Variable::Magic::TestGlobalDestruction::import

Make Variable::Magic::TestGlobalDestruction::import return its result

Output a diagnostic if global destruction is not tested

Only test global destruction on perl 5.13.4+

Or if PERL_VARIABLE_MAGIC_TEST_THREADS is set, for debugging purpose.

Remove taint mode from t/15-self.t

Or Config::Perl::V will never be used.

Better logic for deciding if this perl has DEBUGGING enabled

In particular, it uses Config::Perl::V if it is present.

This is 0.55

Remove test dependencies list in POD

Add support for METHOP ops

This new op class has been introduced with perl commit b46e009d9, which was
published with perl 5.21.5.

This fixes RT #99319.

Don't hardcode the OPc_* values

This simplifies the addition of new op classes.

This is 0.54

Bump copyright year

Add support for copy magic on code prototype clone

This introduces the constant VMG_COMPAT_CODE_COPY_CLONE.

Really make t/35-stash.t pass on 5.21.4

Looks like the core was changed between commit 31d52b17 and the release
of perl 5.21.4.

Use run_perl() from VPIT::TestHelpers

This updates VPIT::TestHelpers to 4fb18792.

Make t/35-stash.t pass on 5.21.4

This is 0.53

Bump copyright year

Make Perl version numbers more readable

Remove obsolete kwalitee workarounds

No tabs please

Stop bundling author tests

Get rid of t/99-kwalitee.t

New versions started to throw warnings if specific environment variables
are not set. This may be appropriate for dist manager users, but not really
for the rest of the world.

Since kwalitee metrics are either obvious or useless, there's no big loss
in removing this test anyway.

Update the bug tracker URL in META after the rt.perl.org upgrade

Typo in POD

Reported by dsteinbrunner@pobox.com.

This fixes RT #86338.

Update VPIT::TestHelpers to e8344578

This is 0.52

Add VMG_COMPAT_SCALAR_NOLEN

Starting from perl 5.17.4, perl no longer calls 'len' magic on scalars.
This new compatibility macro reflects this change.

This fixes RT #80388.

Make t/35-stash.t pass with perl 5.17.4 and above

Since this version, delete on a stash does no longer calls 'store' uvar
magic twice.

Simplify Config loading in Makefile.PL

Don't run the POD spelling test in taint mode

This causes the test to crash on perl 5.12 and below with sensitive
architectures like FreeBSD. Ultimately the root of the issue seems to have
been https://rt.perl.org/rt3/Public/Bug/Display.html?id=60378.

Thanks Slaven Rezic for debugging this.

Update author tests requirements

Add BUILD_REQUIRES to WriteMakefile()

Check destruction order for exceptions thrown in free callbacks