Don't segfault for indirect constructs inside the empty package

The empty package was an old feature that was removed in perl 5.10.

Add support for PERL_OP_PARENT

This was introduced in perl 5.21.2.

Use run_perl() from VPIT::TestHelpers

This updates VPIT::TestHelpers to 4fb18792.

Silence a sign-compare warning

This is 0.31

Add some tests for 'new { __PACKAGE__ }'

Remove profanity

We're professionals (or something).

Handle __PACKAGE__ as object name correctly

__PACKAGE__ cannot appear as the object name for indirect calls, but it
can for direct ones. This could cause wrong reports of indirect calls
when the actual package name appeared later in the source, for instance
in a comment. This is fixed by looking for '__PACKAGE__' in the source
buffer when we already have a hit for the current package name, and by
keeping the first one of those two.

This fixes RT #88428.

Update bundled Test::Leaner to 0.05

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.

This is 0.30

Update the bug tracker URL in META after the rt.perl.org upgrade

This fixes RT #84649.

Partially revert 502e3aac.

Vincent: This fixes things, but I don't know why.

This fixes RT #83806 and RT #83839.

This is 0.29

In string-like envs, take the position to the beginning of the string

This fixes RT #83659.

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 is 0.28

Replace check functions in a thread-safe way

Make indirect_find() more resilient to embedded nuls

Clarify what indirect_find() does by renaming some variables

Properly set and check the line number of method and object tokens

This fixes RT #83450.

Silence an "unused result" compiler warning

This is 0.27

Bump copyright year

Make indirect play nice with Devel::CallParser

Update VPIT::TestHelpers to e8344578

Port module loading in tests to VPIT::TestHelpers

Add BUILD_REQUIRES to WriteMakefile()

Quote version numbers in Makefile.PL

Kill Debian_CPANTS.txt suffixes as well

Update author tests, and add a POD spelling test

Also make sure that the kwalitee test unlinks its byproducts.

Require Module::ExtractUse 0.24 for the Kwalitee test

t/40-threads.t should always run at least one test

No test means failure.

Make sure the POD headings are linkable

Reorder the members of the indirect_op_info struct a bit

This is 0.26

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

Revert "Try to prevent t/50-external.t from crashing on Cygwin"

This reverts commit da77956a4d2f4fbfc4d6c4c2ed78063e5654ac5b.

Test "no indirect 'global'" with threads

Port t/46-stress.t to Test::Leaner

This makes the test suite about 25% faster for a threaded perl.

Port threads test to Test::Leaner and move boilerplate to an helper module

Test::Leaner 0.04 is bundled.

Forbid passing 'hook' and 'fatal' at the same time

Carp is required.

Freshen the documentation

Make sure code examples fit in a 80 columns terminal

A couple of cosmetic code changes

Also forcefully reset the localize hint flag for use indirect

Add 'global' option to 'no indirect'

This enables the module globally.

Stop leaking the SVs stored in the hints structure.

After being clone, they only need one refcount push, which is done at the
end of indirect_clone().

Skip the hint entry vivification test on 5.11

It fails the same way even when indirect is not loaded.

This fixes RT #70618.

This is 0.25

Try to prevent t/50-external.t from crashing on Cygwin

Encode the README file in UTF-8

Remove remaining occurences of $^V

The last argument to hv_fetch is the lvalue flag, not the hash

This fixes RT #69291 on perl 5.8.

This is 0.24

Ignore MYMETA.json

Add META.json

Replace $] by "$]" in tests

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

Test that indirect does not vivify entries in %^H

Test interaction with eval STRING

Revert "Always get the hint directly from %^H"

We should always fetch the hint from the COPs so that we get the right
environment in eval STRING.

This fixes RT #64521 and RT #69291 in 5.10.

This reverts commit 6429fd7545e28fe43dd128c3ce6bb2ccf031e514.

Only declare dMY_CXT when really needed

Clarify our policy towards C++ compilers

Switch to qw<>

Bump copyright year

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