mktables performance improvement

mktables performance improvement

The attached patch got the easiest performance improvements to mktables.
Hopefully this is good enough for now.

This involved:

1) Nicholas' patch
2) I stored complete_name instead of recomputing it each time.
3) Used $_[xxx] instead of shift in very heavily used subroutines
4) removed trace accidentally left in.

I also changed the misspelled subroutine name discovered by Craig Berry.
I searched for any other misspellings and didn't find any.

Also removed trailing white space that keeps creeping back in, and now
this doesn't generate pod entries if not outputting a pod file, and
clarified warning message if no mktables.lst is present.

I couldn't figure out a way to conditionally use 'no overloading', as
it is called at compile time.  So I just commented out the old stuff
that will work for 5.8, with a note about using that if you want to
use 5.8