Reset the callbacks when the root interpreter is destroyed

This fix segfaults when the pragma is first loaded from inside a thread.
The global state for the root interpreter is also freed, so it won't leak
anymore.

Make some room in t/4*.t

c07766a234204d78a80b6d75534e03ae3465b6f3 is a better fix for RT #47902 than 4cb0e2cc26fbe39a7a040b090b1a1e6c2380af0f

As the issue wasn't actually fixed for non-anonsubs.

Only remap the hint through the pointer table for threaded perls

And just use the coderef as the hint when we don't need the require
workaround.

I_THREADSAFE ought to be true when PERL_INDIRECT_PM_DISABLE is set

Use the hint pointer as the unique identifier for the %^H entry

The previous solution (using the callback CV) could cause pragma leaks if
you specified the same "hook" (i.e. a reference to the same sub) in two
different require scopes.

Fix signedness of the pointer<->int conversions

SvIOK and SvROK imply SvOK

This is 0.18

Make sure PERL_INDIRECT_PM_DISABLE is not set before running tests

Add the PERL_INDIRECT_PM_DISABLE environment variable

This is 0.17

Freshen t/40-threads.t

More freshness for t/21-bad.t

Freshen t/10-args.t

Don't use 'Fatal' as a metasyntactic name

Freshen t/30-scope.t

Replace an occurrence of SvREFCNT_inc by SvREFCNT_inc_simple_NN

Turn on CvCLONE for all anonymous subs passed as hooks

So that they're cloned later, which seems to fix some strange issues that arose with a5339a6ced4d5e3ad1541320476c3d6bf8ff9408 when the default callbacks were no longer capturing an external pad (which used to make them eligible for cloning).

This fixes RT #47902.

Fix t/80-regressions.t failing on Win32

The Windows shell doesn't like double quotes inside -e code.

Document and test constant folding

This is 0.16

Cover ck_scope actually handling an OP_SCOPE

This never happens unless taint mode is off.

Remove ck_leave()

We don't need to clean the map entries for leave ops because they can only be created by mutating from a lineseq.

Document the diagnostics

Thanks Andrew and Florian

Freshen t/22-bad-mixed.t

Freshen t/20-good.t

Rename try() to zap() as we'll need try() for something else soon

Simplify the warning handler in t/21-bad.t

Document that we know handle blocks

Handle indirect calls on blocks

Make the default error message public

Only enable the pragma during compile time

In particular, don't handle OPs that are created as part of load_module().
This solves RT #47866.

Revamp t/21-bad.t

This is 0.15

Fix "meth meth" not being correctly reported

Add a stress test

Fix line number for multiline indirect constructs

Store a proper op info instead of hijacking a SV

Factor the cloning logic in a separate function

A few POD nits

Remove unused variable.

This is 0.14

Be proud about our workarounds

Use a better metasyntactic name instead of C<sub>

Don't actually create an entry when storing NULL in the ptable

This is actually unneeded because ptable_fetch() returns NULL when it can't find the entry. This way we can remove the short-lived ptable_delete().

Fix a small leak of hints objects

Prevent bogus invalid syntaxes caused by reallocated memory chunks

Make sure that all our check functions reset a possible previous entry in the pointer table. Then, check in ck_entersub that the object op (the first kid after the pushmark) is of a proper type, so that we are pretty sure that its map entry has been either reset or filled.

This is 0.13

Test the workaround more extensively

Work around the "pragma propagating into require" bug

Rename 'id' to 'hint' for consistency

Stop leaking the package name passed to the hook

We want to test code in ck_entersub(), not hint

Add a TODO test for the "pragma propagating into require" issue

Get rid of t/90-boilerplate.t

Lower the ptable initial size

5.9.5 fix

Also pass the file and the line number to the hook

So that we can warn or die with the correct error message. This is required because caller() can't be trusted at compile time.

Pass copies as hook arguments

Make names stored in the op map readonly

This is 0.12

POD cleanup

Freshen the synopsis

Introduce the 'hook' unimport option

Rename some test files

Disable thread safety for 5.8.x on Windows

Make the op map thread safe

Introduce the I_THREADSAFE() constant

Make the hint thread-safe and store a coderef called each time an indirect construct is spotted

Don't hardcode the package name as the hints key

Switch to a ptable-based OP map

Slashes are reserved and hence must be encoded in the search part of an URL

Tighten the scope of pl_linestr in indirect_map_store()

Test use/no indirect while parsing an indirect construct

Point the repository to the gitweb instead of the bare repo

Don't even try to fetch the object if the method couldn't be found

Somewhat improve the readability of indirect_ck_entersub()

Fix negation precedence

Silence an "unused return value" warning

Consting fix

This is 0.11

Use recent ExtUtils::MakeMaker features to generate a better META.yml

Add a cast to make icc happy

Silence a visibility warning

Fix a possible collisions by hashing pointers with the format %u

This is 0.10

Yet more diagnostics for t/30-scope.t

Bump copyright year

Move the CAVEATS paragraph above DEPENDENCIES

Cleanup indirect_map_clean() recursion

This is 0.09_01

Better handling of extraneous warnings in t/30-scope.t

Phrasing fix

Really fix the "${\(new Hlagh)}" bug by ensuring that the kid of a rv2sv is a SVOP before looking for its sv

In t/30-scope.t, avoid failures for "eval 'no indirect; $x = new Foo'" for being seen as 'more than two warnings' because of an uninitialized $w

This is 0.09

Document the pragma not propagating into 'eval STRING' on 5.8

Skip the segfaulting "${\(new Hlagh)}" test