Update ptable.h

This is 0.23

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.

Ignore MYMETA.yml

Get rid of CALL_FPTR()

Get rid of indirect_is_indirect

Its check is redundant and not decisive at all.

Refine indirect_find

Always get the hint directly from %^H

Since we explicitely ensure that we only fetch the hint at compile time, we
should always be able to get it from %^H. However, the require propagation
bug was fixed for %^H only in 5.12, while it was fixed in 5.10.1 for the
internal hint chain. So we also have to enable the require propagation
workaround on 5.10.1.

Document and test that the "-e without semicolon" bug is fixed on 5.12

Rename t/80-regressions.t to t/50-external.t

Remove some dead code in indirect_ck_method

We already check that the first kid of the method op is a const op. Thus
it must have been checked before. Otherwise, someone has returned a const
op from a custom check function and haven't recalled in the old check
chain, which is bad and ultimately their fault.

Store the offset from the source buffer beginning in the op info object

This fixes potential errors where some indirect constructs are wrongly
reported (or failed to be) when the source buffer is grown between the
method and the object name.

Also, from now on, an op info object is never stored when the correct
position could not have been found in the source buffer. This is really
how it should have behaved from the beginning.

Properly handle method_named ops

They are directly generated (i.e. without being mutated from a method op)
for use foo/no foo. This should solve the random failures of t/30-scope.t.

This is 0.22

Yet more quotelike tests

More quotelike environments testing

Get rid of the linestr check

It simplifies the code, remove some potential breakage if the linestr
changes inside an indirect expression, and allows for reporting indirect
constructs in quotelike environments.

Typos

Allow /^:?fatal$/ to specify the lethal behaviour

This fixes RT #59498.

Only check methods where the method name is a constant

This fixes RT #60378.

This is 0.21

Fix the thread destructor trick for 5.13.1

This fixes RT #57699.

Require perl 5.8.1

5.8.0 seems to cause a lot of "Modification of a a read-only value" errors.

Use a PPCODE: section in CLONE

MY_CXT_INIT can be a declaration, so put it in a block to be strictly conformant to C89

Thanks Jan Dubois for reporting.

Add Tatsuhiko Miyagawa's regression test

Disable taint mode for t/30-scope.t

It prevents a regression test from failing with the old code.

This is 0.20

Remove an unused variable

Fix test failures with 5.12 on Windows where Strawberry Perl crashes because the SystemRoot environment variable is missing

Protect $@ in the regression test for RT #50570

Add a regression test for a bug that appeared while fixing RT #50570 but got re-fixed in a4ebcbe07de579d5877df84c114f162f49019724

Add a regression test for RT #50570

Identify the require scope by the outmost cv outside of the current one

{ENTER,LEAVE}_with_name are no longer needed starting perl 5.11.4

Bump copyright year

Add a link to Matt's blog post

Update ptable.h

Fix for {ENTER,LEAVE}_with_name

Re-indent WriteMakefile() arguments

Build the $name and the main $file from $dist

Put prerequisites in their own separate hash

Give an explicit value to dynamic_config

Only look up to the first require scope for the require propagation workaround

Kwalitee test overhaul

This is 0.19

Introduce I_FORKSAFE

We don't need this (wrong) macro wrapper for indirect_teardown()

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