unicode-org / icu4x / 8487001146

73%

Build Type

push

github

Committed by web-flow

Commit Message

Simplify LocaleExpander::minimize (#4754)

I was reviewing #4752 and I realized that there's a lot of duplicate
code that can be extracted to a helper function, similar to what we do
in the `maximize` routine.

It appears to make the code slightly faster, too, which wasn't my
primary goal but is a nice side-effect.

```
likelysubtags/minimize  time:   [3.8777 µs 3.9019 µs 3.9297 µs]
                        change: [-6.9850% -6.0393% -5.0788%] (p = 0.00 < 0.05)
                        Performance has improved.
Found 5 outliers among 100 measurements (5.00%)
  3 (3.00%) high mild
  2 (2.00%) high severe
```

Coverage Stats

9 of 19 new or added lines in 1 file covered. (47.37%)

3 existing lines in 3 files now uncovered.

51003 of 66850 relevant lines covered (76.29%)

512491.63 hits per line