Checkable::GetAllChildrenInternal: remove redundant emplace call

`checkable` is already added to the set by the insert call above, so calling
emplace for the same checkable doesn't do anything useful and can be removed.
This commit is contained in:
Julian Brost 2025-02-06 15:46:24 +01:00 committed by Yonas Habteab
parent c465f45200
commit 67664ad7b7

View File

@ -223,7 +223,6 @@ void Checkable::GetAllChildrenInternal(std::set<Checkable::Ptr>& seenChildren, i
for (const Checkable::Ptr& checkable : GetChildren()) {
if (auto [_, inserted] = seenChildren.insert(checkable); inserted) {
seenChildren.emplace(checkable);
checkable->GetAllChildrenInternal(seenChildren, level + 1);
}
}