Page MenuHomePhabricator

Fix more "msort()" vs "msortv()" callsites
ClosedPublic

Authored by epriestley on Jun 18 2019, 2:30 PM.
Tags
None
Referenced Files
Unknown Object (File)
Wed, Nov 20, 4:43 PM
Unknown Object (File)
Sat, Nov 16, 5:43 AM
Unknown Object (File)
Fri, Nov 15, 8:22 AM
Unknown Object (File)
Tue, Nov 12, 3:33 AM
Unknown Object (File)
Fri, Nov 8, 3:14 PM
Unknown Object (File)
Oct 16 2024, 6:18 AM
Unknown Object (File)
Sep 25 2024, 2:44 PM
Unknown Object (File)
Sep 15 2024, 1:34 AM
Subscribers
None

Details

Summary

See https://discourse.phabricator-community.org/t/unhandled-exception-when-logging-in-with-mfa/2828. The recent changes to turn msort() on a vector an error have smoked out a few more of these mistakes.

These cases do not meaningfully rely on sort stability so there's no real bug being fixed, but we'd still prefer msortv().

Test Plan

Viewed MFA and External Account settings panels. Did a git grep 'msort(' | grep -i vector for any more obvious callsites, but none turned up.

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable