Extract _DB__grab_control().

I'm a bit unhappy with the interface - I'm planning to extract a class
and an object to instantiate.

Extract _DB__handle_watch_expressions() .

Extract _DB__determine_if_we_should_break() .

[perl5db] More refactoring.

Extract _restore_shared_globals_after_restart.

Extract _restore_globals_after_restart.

[perl5db-refactor] Extract a function.

_restore_options_after_restart() .

[perl5db] Extract a subroutine.

[perl5db-refactor] Refactoring.

[perl5db-refactor] convert to use vars and &s.

[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.