Cool_Tools/ImHex
1
0
Fork 0
mirror of synced 2025-02-09 23:38:27 +01:00
Code Releases Wiki Activity

fix: Potential race condition with sorting in the pattern drawer

Browse Source
This commit is contained in:
WerWolv 2024-07-01 23:36:17 +02:00
parent c2f661f021
commit dd8e7025d0
1 changed files with 20 additions and 12 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

10
plugins/ui/source/ui/pattern_drawer.cpp
View File

@ -209,6 +209,12 @@ namespace hex::ui {
void PatternDrawer::updateFilter() { void PatternDrawer::updateFilter() {
m_filteredPatterns.clear(); m_filteredPatterns.clear();
if (m_filter.path.empty()) {
m_filteredPatterns = m_sortedPatterns;
return;
}
std::vector<std::string> treePath; std::vector<std::string> treePath;
for (auto &pattern : m_sortedPatterns) { for (auto &pattern : m_sortedPatterns) {
traversePatternTree(*pattern, treePath, [this, &treePath](auto &pattern){ traversePatternTree(*pattern, treePath, [this, &treePath](auto &pattern){
@ -1132,12 +1138,13 @@ namespace hex::ui {
return true; return true;
} }
if (!m_favoritesUpdateTask.isRunning()) {
sortedPatterns.clear(); sortedPatterns.clear();
std::transform(patterns.begin(), patterns.end(), std::back_inserter(sortedPatterns), [](const std::shared_ptr<pl::ptrn::Pattern> &pattern) { std::transform(patterns.begin(), patterns.end(), std::back_inserter(sortedPatterns), [](const std::shared_ptr<pl::ptrn::Pattern> &pattern) {
return pattern.get(); return pattern.get();
}); });
std::sort(sortedPatterns.begin(), sortedPatterns.end(), [this, &sortSpecs](const pl::ptrn::Pattern *left, const pl::ptrn::Pattern *right) -> bool { std::stable_sort(sortedPatterns.begin(), sortedPatterns.end(), [this, &sortSpecs](const pl::ptrn::Pattern *left, const pl::ptrn::Pattern *right) -> bool {
return this->sortPatterns(sortSpecs, left, right); return this->sortPatterns(sortSpecs, left, right);
}); });
@ -1148,6 +1155,7 @@ namespace hex::ui {
} }
sortSpecs->SpecsDirty = false; sortSpecs->SpecsDirty = false;
}
return true; return true;
} }