[PR #16788] A minor cleanup of ProfileViewModel #30993

Open
opened 2026-01-31 09:44:25 +00:00 by claunia · 0 comments
Owner

Original Pull Request: https://github.com/microsoft/terminal/pull/16788

State: closed
Merged: Yes


This is just a minor cleanup I did as a drive-by while working on
customized font fallback. The benefit of this change is that it's
a tiny bit less expensive, but also that it's a lot easier to read.
The split into "get index" and "get string by index" helps us to
more easily handle both, missing locales and locale fallback.
The code that ties everything together then ends up being just 7 lines.

**Original Pull Request:** https://github.com/microsoft/terminal/pull/16788 **State:** closed **Merged:** Yes --- This is just a minor cleanup I did as a drive-by while working on customized font fallback. The benefit of this change is that it's a tiny bit less expensive, but also that it's a lot easier to read. The split into "get index" and "get string by index" helps us to more easily handle both, missing locales and locale fallback. The code that ties everything together then ends up being just 7 lines.
claunia added the pull-request label 2026-01-31 09:44:25 +00:00
Sign in to join this conversation.
No Label pull-request
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: starred/terminal#30993