Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Refactor: Completely remove UnsortedNamedInts #2656

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

MattHag
Copy link
Collaborator

@MattHag MattHag commented Nov 1, 2024

Simplify the code by removing this unnecessary class. Replace with default NamedInts implementation.

Related #2273

Simplify the code by removing this unnecessary class. Replace with
default NamedInts implementation.

Related pwr-Solaar#2273
@pfps
Copy link
Collaborator

pfps commented Nov 2, 2024

I think the non-sorted version was at least at one time needed to get ordering of options in the UI correct so this PR needs to be checked to ensure that it does not change the UI.

@MattHag
Copy link
Collaborator Author

MattHag commented Nov 2, 2024

Can be, altough it sounds counterintuitive. If some natural order should be enforced, then it's better to add an explicit function that does that.

@pfps
Copy link
Collaborator

pfps commented Nov 2, 2024

As I remember, the order doesn't correspond to any sorting. It is just the order in which the items are added to the collection. But I'll have to check out how the UI works for several settings.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants