un-break UCA tool: Adlam new FracUCA byte after Garay #616
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
In the CLDR/ICU FractionalUCA.txt, the addition of the new script Garay between Medefaidrin and Adlam pushed Adlam across a primary-weight lead byte boundary, which does not work for a primary-compressible script. I needed to split the sequence of scripts from Vai to Adlam across two lead bytes and decided to simply start a lead byte with Adlam.
Some of these scripts are not used widely but use a fair bit of primary weight space because they are cased, and thus stored with two-byte primary weights, so that their collation elements easily fit into 32 bits, instead of with three-byte primaries.
The CollationTest file diffs are large and hard to look at. I don't expect much of a review there.