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

Update how author tests load their dependencies

Also make sure that the kwalitee test unlinks its byproducts.

This is 0.51

Documentation tweaks

In particular, document how 'free' callbacks propagate exceptions.

Properly propagate exceptions when a free callback dies at the end of eval

Kill Debian_CPANTS.txt suffixes as well

Make the 'reset RMG flag' workaround thread-safe

This uses trampoline ops for threaded perls, where it is not allowed to
modify the contents PL_op points to, but just what PL_op points at.

Reset the SV each time for the 'reset RMG flag' workaround

This bug caused the workaround to only update the flag of the first hash
that it was called for, regardless of subsequent calls with different
hashes.

Remove an unused variable

Prettify the default no-op callback

Don't crash if dispell is called in a clear, free or uvar callback

Rename vmg_uvar_del() to vmg_mg_del() to reuse its logic

The new vmg_mg_del() is now also used to remove our PERL_MAGIC_ext tokens.

Make sure all entries of MY_CXT.b__op_stashes are initialized

Otherwise CLONE may be testing uninitialized memory areas.

Fix indentation in vmg_call_sv()

The previous commit just made it obvious that it was misindented.

Port the svt_free-specific part of vmg_call_sv() to a customizable hook

Always enable VMG_SAVE_LAST_CX

This allows to remove the special case of only setting in_eval when we are
in a destructor.

Make the uvar user data into a proper struct

This is 0.50

Improve Capture::Tiny discovery in t/17-ctl.t

Don't use the main:: stash in the last test of t/17-ctl.t

As it is less subject to extra lookups.

This fixes RT #77991 in a better way.

Consistently format magic types with I<>

Remove a couple of commented asserts

Leak less memory when an exception is thrown from a free callback

Move the global destruction logic of t/15-self.t out in an helper module

Lower t/17-ctl.t's last test expectations

With ActivePerl, there's only one line produced by this test because of
some strange interference between local *_, loading Exporter.pm and
compiling '-d _'. Since this test has also been broken recently by a
change in the core, it is more reliable to stop asking for the exact
count of matches.

This fixes RT #77991.

Don't leak vtables and wizards freed during global destruction

At this time, we cannot access the callbacks, but we are still allowed
to call Safefree() and PerlMemShared_free().

Cleaner version numbers

Remove one extra mention to MGf_COPY

Also explicitely state from which perl versions the MGf_* constants are
available.

Jumbo POD overhaul

Lots of rewordings and clarifications.

Teach t/35-stash.t about perl 5.17.1

This is 0.49

Teach t/17-ctl.t about perl 5.17.0

Reuse the diag() helper in the TestThreads module

Require Module::ExtractUse 0.24 for the Kwalitee test

This is 0.48

Decorate the magic callbacks section

Remove an useless #define wrap

Bump copyright year

Allow passing ref-to-undef as callbacks to install a noop callback

In t/22-len.t, localize the wizard for strings to the SKIP block

Move the sv length logic into a new vmg_sv_len() function

Protect $@ inside Variable::Magic::TestWatcher::init_watcher

Test that unqualified sub names as callbacks default to the current package

This is 0.47

Test 'delete' uvar magic and introduce VMG_COMPAT_HASH_DELETE_NOUVAR_VOID

'store' and 'delete' uvar magics don't need the uvar/clear hack

Clean up vmg_uvar_val() a bit

Minimize the accesses to the optree during the uvar/clear hack

Prettify the declaration of vmg_svt_copy()

vmg_cast() should set mg_flags according to the vtable

And not according to the Perl callbacks.

Make the watch { } wrapper properly apply context

The magic signatures are no longer needed

Enforce saner checks on wizard SVs and MAGIC tokens

This commit adds a variety of helper functions that extract the vmg_wizard
object from a SV or a MAGIC token. Version with (for direct interaction
with the user) and without checks (for magic callbacks, where we know the
MAGIC token is already valid) are provided.

Reorder some code

Rename vmg_wizard_* to vmg_wizard_sv_* and {MGWIZ,mgwiz}_* to vmg_wizard_*

Clone wizard objects through dup magic

ptable.h is no longer needed.

Clean up threads tests a bit

Share the vtables with threaded perls

When a magical variable is cloned into a new thread, the associated vtables
aren't cloned by mg_dup(), but simply shared. This means that if a wizard
is casted upon a variable in the main interpreter, then a thread is created,
then the wizard is destroyed, and then the cloned variant of the magical
variable is manipulated, perl can no longer access the vtable (it was freed
when the wizard was destroyed) and memory misreads happen.

To solve this, the only solution is to make the vtables associated with
wizards shared and reference-counted.

Wrap MUTEX_{LOCK,UNLOCK} in macros so can we don't have to #ifdef them

MGf_COPY and MGf_DUP should always be set for the perls we consider

Abstract the MGWIZ creation into a new vmg_mgwiz_alloc() function

Pass the 'data' callback after the 'op_info' flag when calling _wizard()

Threads tests may not be able to spawn all the threads

To work around this :
- move all the threads boilerplate to a new helper test module ;
- capture the warnings/errors that threads->create may throw and reprint
them properly as diagnostics ;
- don't set a plan in threads tests.

Remove trailing whitespace