On 04/10/2019 20:55, Roger Riggs wrote:
Hi Andrew,
That does seem more straightforward.
The name ALWAYSCOMPACTABLE doesn't really match the rest of the
nomenclature used in the compact string code.
Can you change the name to LATIN1COMPATIBLE?
Its similar to the ASCIICOMPATIBLE case and tied in to the Latin1
coding for used
in StringCoding.
I hope another Reviewer will take a look also.
I skimmed through the webrev and I think the approach looks good (and
much better than the original suggestion to scan byte array when
creating the Decoder).
-Alan