Replace the InOrderCounter in the ATS with something more sensible #10044

Closed
opened 2026-01-31 02:11:07 +00:00 by claunia · 2 comments
Owner

Originally created by @leonMSFT on GitHub (Aug 6, 2020).

Inside of CommandPalette.cpp's filtering function lives a counter whose sole purpose is to guarantee a stable sort when the WeightedCommands are thrown into the priority queue. This definitely should be replaced with something more pleasing.

Originally created by @leonMSFT on GitHub (Aug 6, 2020). Inside of `CommandPalette.cpp`'s filtering function lives a `counter` whose sole purpose is to guarantee a stable sort when the `WeightedCommands` are thrown into the priority queue. This definitely should be replaced with something more pleasing.
claunia added the Issue-TaskNeeds-Tag-FixProduct-TerminalArea-UserInterface labels 2026-01-31 02:11:07 +00:00
Author
Owner

@Don-Vito commented on GitHub (Nov 16, 2020):

@leonMSFT, @zadjii-msft - I guess this one can be closed as weighted command exists no more.

@Don-Vito commented on GitHub (Nov 16, 2020): @leonMSFT, @zadjii-msft - I guess this one can be closed as weighted command exists no more.
Author
Owner

@zadjii-msft commented on GitHub (Dec 15, 2021):

Oh, wow yea it totally can. Must've missed this one!

@zadjii-msft commented on GitHub (Dec 15, 2021): Oh, wow yea it totally can. Must've missed this one!
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: starred/terminal#10044