[perl5db] Extract a function.

[perl5db] Remove a stray declaration.

[perl5db-refactor] Extract a method.

[perl5db-refactor] Extract _delete_all_breakpoints.

[perl5db] Remove leading ampersands in sub calls.

More refactoring.

[perl5db] Remove ampersand.

[perl5db-refactor] Merge two conditionals.

Add more.

Refactored a loop.

[perl5db] Avoid predeclaration.

[perl5db] Refactored more.

[perl5db-refactor] Remove leading ampersands.

From subroutine calls.

[perl5db] refactoring + minor bug-fix.

The regex h\s* matches for every string that starts with "h". It should
match only if h is the only thing so a \z is needed.

[perl5db-refactoring] Refactored parse_options.

[perl5db] Convert $| to ->autoflush(1).

[perl5db] Refactor the LineInfo function.

[perl5db-refactoring] Got rid of trailing whitespace.

[perl5db-Refactor] Convert $_ to a lexical.

[perl5db] Extract a function.

[perl5db] Remove some trailing space.

[perl5db] More refactoring.

1. Got rid of a trailing block comment because the block is shorter.

2. Now using something more sensible than s/^/perl/.

[perl5db] Convert @pods to a hash and moved it out.

This is part of the refactoring.

Move away from more "or do { ... }" statements.

[perl5db] Refactored do {...}s into ifs.

[perl5db] More refactoring.

1. Made the POD better.

2. Converted more && do to ifs.

3. More eq conversions.

4. Captures in lexical variables.

5. Using /ms instead of [\x00-\xFF].

More refactoring.

Convert some COND() && do to ifs.

Also converted "^" and "$" to the less ambiguous \A and \z and converted
an /^q$/ to the "eq" operator.

Here I am starting to refactor the perl debugger.

Add a test for the frame option.

[perl5db] o PrintRet=0 in void context.

Test o PrintRet=1 in void context.

[perl5db] Add another test.

[perl5db] Test o PrintRet in list context.

[perl5db] add a test for o PrintRet=0.

Add a test for o PrintRet.

Test o inhibit_exit=0.

Fix a bug with o AutoTrace.

There it was.

[perl5db] Add a test for t command with sub calls.

[perl5db] Test o AutoTrace .

It appears to work fine now. Strange.

[perl5db] Add a test for the 't' command.

Apparently, the 'o AutoTrace' command is broken since perl-5.16.x and I
want to get a good output for 't' first before fixing it.

[perl5db] Add a test for warnLevel=1.

[perl5db] Add test for dieLevel option.

[perl5db] Test the recallCommand option.

[perl5db] Test the M command.

Test more for the m staetement.

Test the m command.

[perl5db] Test the = command.

Correct the test blurbs.

Add a test for the H command.

[perl5db] Add a test for H -7.

[perl5db] Fix source cmd from typeahead.

With a test.

Add a test for the source command.

Extract a common subroutine.

Rename to something more meaningful.

Convert to DebugWrap.

Convert more to DebugWrap.

Convert the 61222 test to DebugWrap.

Extract _calc_threads_wrapper.

Remove some no-longer used code.

Prepend with an underscore.

To make it an internally-used subroutine.

Convert another test to DebugWrap.

Convert another test to DebugWrap.

Convert another test to DebugWrap.

Convert another test to DebugWrap.

Extract a constructor.

This will help in converting the other instances to DebugWrap.

Convert a test to DebugWrap.

Convert a perl5db.t test to DebugWrap.

Add a test for l -num.

Test the ! command.

Test the { * command.

Convert to a different quoting.

This was done so gvim won't be confused with the bracket-matching.
I'll report the problem to the perl.vim project.

Test for the { and { ? commands.

Add a test for the < and > commands together.

rmv context from Perl_croak_no_modify and Perl_croak_xs_usage

Remove the context/pTHX from Perl_croak_no_modify and Perl_croak_xs_usage.
For croak_no_modify, it now has no parameters (and always has been
no return), and on some compilers will now be optimized to a conditional
jump. For Perl_croak_xs_usage one push asm opcode is removed at the caller.
For both funcs, their footprint in their callers (which probably are hot
code) is smaller, which means a tiny bit more room in the cache. My text
section went from 0xC1A2F to 0xC198F after apply this. Also see
http://www.nntp.perl.org/group/perl.perl5.porters/2012/11/msg195233.html .

PADRANGE: don't assume targs are contiguous

Normally in something like my($x,$y,$z), the 3 targs are contiguous;
however eternal modules like Function::Parameters can break that
assumption, so don't assume it.

fix hash order dependency bug in new Module::Corelist::deprecated_in()

The code was doing

    $version= List::Util::min(keys %$versions);

which introduces a hash order dependency as min() returns either the first or last
(i didnt check which) of the lowest items in the list, and considers 5.011000 and
5.011 to be equivalent. Depending on the hash order it would return
either one. Hash randomization revealed this bug immediately.

Changing the function to use List::Util::minstr() eliminates the
dependency.

    $version= List::Util::minstr(keys %$versions);

[perl #43077] Make goto &sub leave @_ alone

It is a little tricky, as we have to hang on to @_ while unwinding the
effects of local @_.

leakfinder.pl: Many more exceptions

Env.pm: Fix splice in list context

It was passing an extra argument to splice, causing splice @PATH, 0, 0
to append ":0" to the environment variable.

Increase $Env::VERSION to 1.04

Even more regexp charclass leaks

toke.c: White-space only; no code changes

This indents a newly formed block, and changes one other line to line up
better.

toke.c: Don't force \N{} into utf8 unnecessarily

regcomp.c no longer requires everything containing \N{} to be in UTF-8.
I'm not sure of the exact commit; it might even have been in 5.16.  But
it was done by the time 86e88272fdabc40e3b168a3cc43af5e86284d01b was
done.

Therefore we can remove the temporary code that forced utf8, and replace
it with code that handles the non-utf8 case.

Note that outside patterns, \N{} still forces utf8.  This is so that
Unicode semantics will be imposed on the string it resides in, no matter
how it is used.  Patterns have a flag that indicates Unicode semantics,
so don't need to be in utf8.

toke.c: Remove redundant checks

This code is redundant, as the whole string has been checked earlier for
valid utf8ness, or we assume that the internal Perl handler, charnames,
is correctly returning valid utf8.

toke.c: Reorder declaration

This is in preparation for the next commit.

toke.c: Use UTF-8 macros for portability

I noticed this code in toke.c that hand-rolled its own UTF-8 handling,
and is assuming an ASCII platform.  There are macros that do the same
thing that this code does, and are portable.

utf8.h: Add macro that handled malformed 2-byte UTF-8

The macro used traditionally to see if there is a two-byte UTF-8
sequence doesn't make sure that there actually is a second byte
available; it only checks if the first byte indicates that there is.
This adds a macro that is safe in the face of malformed UTF-8.

I inspected the existing calls in the core to the unsafe macro, and I
believe that none of them need to be converted to the safe version.

util.c: Add some asserts()

These functions improperly have a signed parameter, when the value
should always be at least 0.  We have a field report that one of them is
getting called with a negative value.  Add asserts to catch this
condition (if compiling assertions is on) earlier.

utf8.c: Remove redundant unlikely branches

The 2 lines removed in each function provide an early exit if the input
is malformed UTF-8.  Other code executed later makes the same test.
But most inputs are going to be well-formed, so the test will almost
always fail, so will slow things up.

Validate above-Latin1 characters in \N{} aliases

This completes the process of allowing users to define their own aliases
for \N{} in any language they choose.  Names have some validation
applied so that they can't, for example, begin with something that is a
digit in some Unicode script.  Tests and documentation are included in
this patch.  The loop in toke.c that does the validation for
user-supplied translators is revamped, and the messages that are output
when there is an error are fixed to work with UTF-8.

embed.fnc: Allow toke.c to call core_swash_init()

This internal function is allowed to just a few core functions
by #ifdef's in embed.fnc.  Expand the list to include toke.c, as it will
be needed in a future commit.

toke.c: Fail on malformed UTF-8 in \N{} input

The handler for \N{} can be user-supplied and charnames itself
shouldn't have to worry about malformed input.  This changes toke.c to
check for malformed input before calling the \N{} handler.

toke.c: Remove no-longer needed {...} and outdent correspondingly

No code changes

toke.c: [RT#73022] Make \N{...} UTF-8 clean.

toke.c Refactor S_get_and_check_backslash_N_name()

This code was recently factored out into a separate subroutine, and was
originally designed for a non-fatal deprecated warning.  This
refactoring just goes immediately to failure when an illegal character
is found.

(It also changes the code to use Perl standard coding practices)

perldiag: Add entry for unknown charname

toke.c: Don't validate core charnames output

charnames now refuses to allow invalid aliases to be installed, so we
only need to validate those that come from a custom translator.

charnames: Don't accept illegal :aliases

Now that improper names for characters are an error, we can forbid them
at definition time.  For the time being allow a colon in the check that
continues to be run in toke.c.  This will be removed in a future commit.

regen/mk_PL_charclass.pl: Use mktables table for charname

This commit uses the mktables defined table for whether or not a
character is a legitimate charname continuation.  This will allow it to
be kept in sync with other code that needs the definition.

The only change this makes is to delete "colon" from being a legitimate
continuation character.  A colon was only accepted because it was used
in the paradigm for like "Greek: Alpha", and is not part of any
actual character name.

mktables: Create tables for charname beginning and follow-on

mktables is changed to add two new tables, one that matches the first
character in a character names, and one that matches continuation
characters.