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

Prettify the constants export list

5.13.2 calls get magic on globs

Add the new compatibility macro VMG_COMPAT_GLOB_GET to reflect this change.

Fix CLONE_PARAMS struct initialization for perl 5.13.2

Always use a safe version of call_sv()

All the relevant code has been factored out of vmg_cb_free() into a new
vmg_call_sv() function.

Exception propagation fixes

First, make sure that exceptions thrown in a free callback at the end of a
regular block are propagated.

Second, prevent duplication of error messages when dieing in a free callback
at BEGIN time.

This is 0.42

Fix tests for perl 5.13

Fix t/18-opinfo.t failures with perl 5.13

Complete coverage of the len callback

Improve op_info coverage

This is 0.41

Compatibility fix for 5.12.0

A few doc corrections

Tune the userdata example

Correctly propagate the errors thrown when variable destruction happens at compile-time

This is 0.40

Indentation

Bump copyright year

Use SvREFCNT_inc_simple_void() whenever possible

Use SvRV_const() whenever possible

SvPV_const is only available since 5.9.3

Cast the op_info value before applying bit shifts

Also move the opinfo member before the uvar one in the MGWIZ struct, as
it's used more frequently.

Use size_t as the index type in the pointer table

Fix expected constness for utf8_length() arguments

Fix type coercions on values returned from callbacks

call_sv() ought to take an I32

Explicitely cap the op_info parameter to 255

Wizard ids are IV, not UV

Fix memory miswrite when passing data arguments to cast()

This is 0.39

Document passing a string reference as the callback

Fix compiler warnings with the Intel CC

Test string callbacks

Test destruction of magic callbacks

Also test getdata() in t/80-leaks.t

Stop leaking objects stored in the data slot

Stop passing arguments to the data callback through an AV