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

Make sure code examples fit in a 80 columns terminal

Preserve the PATH environment variable when running a sub-perl on cygwin

Silence an "unused" compiler warning

Tweak some optional module loading in tests

Add META.json

Encode the README file in UTF-8

Replace $] by "$]"

An old bug could cause $] to be initialized to a wrong decimal value.
Since its stringified value was always right, we use it instead.

Occurences of $^V were also replaced by $].

Ignore MYMETA.json

Update vmg_opclass() for perl 5.15

Even if magic is no longer called by the aelem(fast).

This is 0.46

Switch to qw<>

Update magical flags after dispelling magic

This fixes RT #64866.

Bump copyright year

Tighten the scope of two variables unused on 5.8

This is 0.45

A terser way to display versions for modules used in tests

Only use Capture::Tiny when it is working on a simple example

A bare glob that isn't part of a fetch doesn't call get magic anymore in 5.13.7

Add more glob tests

Wording fix

Give a saner default to pPTBLMS

This silences a compiler warning.

Skip NULL values when walking a table

ptable_fetch already returns NULL for nonexistent entries, so NULL can't
possibly have any other meaning than "this node is not currently in use".

Add ptable_delete to ptable.h

Explicitely state that C++ compilers aren't supported

Ignore MYMETA.yml

Clarify minimum perl 5.10.0 requirement for hash magic

This is 0.44

Correctly propagate exceptions from _wizard()

Only load Carp.pm when throwing an error

Describe why we don't have to guard against PL_dirty in vmg_mgwiz_free()

Force threads tests with the PERL_VARIABLE_MAGIC_TEST_THREADS environment variable

Skip threads tests unless perl version is 5.13.4 or greater

There was a long standing bug in the handling of GV <-> CV double linkage
that could (and explicitely did with a poisonous perl) cause segfaults at
thread destruction. It got fixed by Dave in commit 803f274 which went in
5.13.3, but the fix was amended for 5.13.4 in commit 09aad8f. Since it's
not really fair for the user to not be able to install the module because
of this, we skip the threads tests unless perl is at least 5.13.4.

Remove the patchlevel test

Commit d29a774d106d145036eda849ab57f42410100008 removed the only place it
was used, and perl left perforce almost three years ago.

Remove the test for non-released perl 5.11.0

Who would be crazy enough to still use them? And the test seems to break
with some git checkouts.

Force linking against the perl dll when using gcc 3.4 on Windows

When a Windows perl links an XS shared object to an import library using
gcc/g++ version 3.4.x, strange breakage happens at load time because the
mutex-related symbols seems not to be reachable. This situation happens
especially for AS perl and the default compiler suite it installs.

We fix this by forcing g++ to link against the dll instead, which should
be in the same directory as the perl executable (at least for AS perl).

This (should) fix RT #51483.

Clarify some logic in Makefile.PL

SvREFCNT_dec already checks if the SV is non null

This is 0.43

More control tests