CLEANUP(contentmodel): Use ranges::find_if instead of a loop

OpenMW · Jun 23, 2024 · fbdc4f4 · fbdc4f4
1 parent 40aee53
commit fbdc4f4
Showing 1 changed file with 3 additions and 8 deletions.
diff --git a/components/contentselector/model/contentmodel.cpp b/components/contentselector/model/contentmodel.cpp
@@ -1,6 +1,7 @@
 #include "contentmodel.hpp"
 #include "esmfile.hpp"
 
+#include <algorithm>
 #include <fstream>
 #include <memory>
 #include <stdexcept>
@@ -672,14 +673,8 @@ void ContentSelectorModel::ContentModel::setNonUserContent(const QStringList& fi
     for (auto* file : mFiles)
         file->setFromAnotherConfigFile(mNonUserContent.contains(file->fileName().toLower()));
 
-    int insertPosition = 0;
-
-    for (auto* file : mFiles)
-    {
-        if (!file->builtIn())
-            break;
-        ++insertPosition;
-    }
+    int insertPosition
+        = std::ranges::find_if(mFiles, [](const EsmFile* file) { return !file->builtIn(); }) - mFiles.begin();
 
     for (const auto& filepath : fileList)
     {