maxmustermann/terminal
1
0
Fork 0
Code Issues Pull requests Projects Releases Wiki Activity

use an actually appropriate std method instead of something exotic like stable_partition

Browse source
This commit is contained in:
Schuyler Rosefield 2021-09-16 16:17:39 -04:00
parent be2cdbe60a
commit d95629be86
1 changed files with 3 additions and 5 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

8
src/cascadia/Remoting/Monarch.cpp
View file

@ -588,13 +588,11 @@ namespace winrt::Microsoft::Terminal::Remoting::implementation
{
std::unique_lock uniqueLock{ _mruPeasantsMutex };
// I wish I had std::erase_if (C++20) but I don't yet
// partition the array into [not dead | dead ]
auto partition = std::stable_partition(_mruPeasants.begin(), _mruPeasants.end(), [&](const auto& p) {
auto partition = std::remove_if(_mruPeasants.begin(), _mruPeasants.end(), [&](const auto& p) {
const auto id = p.PeasantID();
const auto it = std::find(peasantsToErase.cbegin(), peasantsToErase.cend(), id);
// keep the element if it was not found in the list to erase.
return it == peasantsToErase.cend();
// remove the element if it was found in the list to erase.
return it != peasantsToErase.cend();
});
// Remove everything that was in the list