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

Reply via email to