APItest/t/utf8_warn_base.pl: Revamp loop to/from utf8
authorKarl Williamson <khw@cpan.org>
Sun, 2 Jul 2017 15:11:17 +0000 (09:11 -0600)
committerKarl Williamson <khw@cpan.org>
Thu, 13 Jul 2017 03:14:24 +0000 (21:14 -0600)
commit69485e19957cdc18073e734041cf0c35515c0995
tree7fbfea986752ef886899e60f9ab9344000e3df70
parent601e92f1ff871a52b4fbb83b5061574a3541c8f3
APItest/t/utf8_warn_base.pl: Revamp loop to/from utf8

This test file had gotten kinda messy as new tasks were shoe horned into
it.  This cleans it up, and positions it to be easier maintain going
forward.  I tried to minimize the number of changes shown per commit,
but this is the minimal I could get, and since it is a revamp, there are
lots of differences.

Some combinatorial explosion has been removed.

A new subroutine is created which compares the expected vs actually
gotten warnings, and is called in two places, removing duplicated code.

This exposed a bug in very large, hence rare, code points.  It will be
fixed in the next commit.  It was far easier to just make all similar
tests TODO here, removing that in the next commit.  This means this
commit has many passing TODOs
ext/XS-APItest/t/utf8_warn_base.pl