Use pathhelpers to populate Collections

apps/components_tests/esm3/readerscache.cpp

@@ -42,7 +42,7 @@ namespace
         const Files::PathContainer mDataDirs{ { std::filesystem::path{ OPENMW_DATA_DIR } } };
         const Files::Collections mFileCollections{ mDataDirs };
         const std::string mContentFile = "template.omwgame";
-        const std::filesystem::path mContentFilePath = mFileCollections.getCollection(".omwgame").getPath(mContentFile);
+        const std::filesystem::path mContentFilePath = mFileCollections.getCollection("omwgame").getPath(mContentFile);
     };
 
     TEST_F(ESM3ReadersCacheWithContentFile, shouldKeepOpenReleasedOpenReader)

apps/niftest/niftest.cpp

@@ -190,8 +190,8 @@ void readVFS(std::unique_ptr<VFS::Archive>&& archive, const std::filesystem::pat
     if (!archivePath.empty() && !isBSA(archivePath))
     {
         const Files::Collections fileCollections({ archivePath });
-        const Files::MultiDirCollection& bsaCol = fileCollections.getCollection(".bsa");
+        const Files::MultiDirCollection& bsaCol = fileCollections.getCollection("bsa");
-        const Files::MultiDirCollection& ba2Col = fileCollections.getCollection(".ba2");
+        const Files::MultiDirCollection& ba2Col = fileCollections.getCollection("ba2");
         for (const Files::MultiDirCollection& collection : { bsaCol, ba2Col })
         {
             for (auto& file : collection)

apps/openmw/mwworld/worldimp.cpp

@@ -32,6 +32,7 @@
 #include <components/misc/constants.hpp>
 #include <components/misc/convert.hpp>
 #include <components/misc/mathutil.hpp>
+#include <components/misc/pathhelpers.hpp>
 #include <components/misc/resourcehelpers.hpp>
 #include <components/misc/rng.hpp>
 
@@ -2885,9 +2886,7 @@ namespace MWWorld
         int idx = 0;
         for (const std::string& file : content)
         {
-            const auto filename = Files::pathFromUnicodeString(file);
+            const Files::MultiDirCollection& col = fileCollections.getCollection(Misc::getFileExtension(file));
-            const Files::MultiDirCollection& col
-                = fileCollections.getCollection(Files::pathToUnicodeString(filename.extension()));
             if (col.doesExist(file))
             {
                 gameContentLoader.load(col.getPath(file), idx, listener);

components/esmloader/load.cpp

@@ -18,6 +18,7 @@
 #include <components/files/conversion.hpp>
 #include <components/files/multidircollection.hpp>
 #include <components/loadinglistener/loadinglistener.hpp>
+#include <components/misc/pathhelpers.hpp>
 #include <components/misc/resourcehelpers.hpp>
 #include <components/misc/strings/lower.hpp>
 
@@ -211,21 +212,20 @@ namespace EsmLoader
         {
             ShallowContent result;
 
-            const std::set<std::string> supportedFormats{
+            const std::set<std::string_view, Misc::StringUtils::CiComp> supportedFormats{
-                ".esm",
+                "esm",
-                ".esp",
+                "esp",
-                ".omwgame",
+                "omwgame",
-                ".omwaddon",
+                "omwaddon",
-                ".project",
+                "project",
             };
 
             for (std::size_t i = 0; i < contentFiles.size(); ++i)
             {
                 const std::string& file = contentFiles[i];
-                const std::string extension
+                const std::string_view extension = Misc::getFileExtension(file);
-                    = Misc::StringUtils::lowerCase(Files::pathToUnicodeString(std::filesystem::path(file).extension()));
 
-                if (supportedFormats.find(extension) == supportedFormats.end())
+                if (!supportedFormats.contains(extension))
                 {
                     Log(Debug::Warning) << "Skipping unsupported content file: " << file;
                     continue;

components/files/collections.cpp

@@ -2,7 +2,6 @@
 #include "conversion.hpp"
 
 #include <components/misc/strings/algorithm.hpp>
-#include <components/misc/strings/lower.hpp>
 
 namespace Files
 {

components/files/collections.hpp

@@ -15,8 +15,7 @@ namespace Files
         ///< Directories are listed with increasing priority.
         Collections(const Files::PathContainer& directories);
 
-        ///< Return a file collection for the given extension. Extension must contain the
+        ///< Return a file collection for the given extension.
-        /// leading dot
         const MultiDirCollection& getCollection(std::string_view extension) const;
 
         std::filesystem::path getPath(std::string_view file) const;

components/files/multidircollection.cpp

@@ -22,7 +22,8 @@ namespace Files
             {
                 const auto& path = dirIter.path();
 
-                if (!Misc::StringUtils::ciEqual(extension, Files::pathToUnicodeString(path.extension())))
+                std::string ext = Files::pathToUnicodeString(path.extension());
+                if (ext.size() != extension.size() + 1 || !Misc::StringUtils::ciEndsWith(ext, extension))
                     continue;
 
                 const auto filename = Files::pathToUnicodeString(path.filename());
@@ -41,7 +42,7 @@ namespace Files
                 {
                     // handle case folding
                     mFiles.erase(result->first);
-                    mFiles.insert(std::make_pair(filename, path));
+                    mFiles.emplace(filename, path);
                 }
             }
         }

components/translation/translation.cpp

@@ -2,6 +2,8 @@
 
 #include <fstream>
 
+#include <components/misc/pathhelpers.hpp>
+
 namespace Translation
 {
     Storage::Storage()
@@ -11,25 +13,24 @@ namespace Translation
 
     void Storage::loadTranslationData(const Files::Collections& dataFileCollections, std::string_view esmFileName)
     {
-        std::string esmNameNoExtension(Misc::StringUtils::lowerCase(esmFileName));
+        std::string_view esmNameNoExtension = Misc::stemFile(esmFileName);
-        // changing the extension
-        size_t dotPos = esmNameNoExtension.rfind('.');
-        if (dotPos != std::string::npos)
-            esmNameNoExtension.resize(dotPos);
 
-        loadData(mCellNamesTranslations, esmNameNoExtension, ".cel", dataFileCollections);
+        loadData(mCellNamesTranslations, esmNameNoExtension, "cel", dataFileCollections);
-        loadData(mPhraseForms, esmNameNoExtension, ".top", dataFileCollections);
+        loadData(mPhraseForms, esmNameNoExtension, "top", dataFileCollections);
-        loadData(mTopicIDs, esmNameNoExtension, ".mrk", dataFileCollections);
+        loadData(mTopicIDs, esmNameNoExtension, "mrk", dataFileCollections);
     }
 
-    void Storage::loadData(ContainerType& container, const std::string& fileNameNoExtension,
+    void Storage::loadData(ContainerType& container, std::string_view fileNameNoExtension, std::string_view extension,
-        const std::string& extension, const Files::Collections& dataFileCollections)
+        const Files::Collections& dataFileCollections)
     {
-        std::string fileName = fileNameNoExtension + extension;
+        std::string fileName(fileNameNoExtension);
+        fileName += '.';
+        fileName += extension;
 
-        if (dataFileCollections.getCollection(extension).doesExist(fileName))
+        const Files::MultiDirCollection& collection = dataFileCollections.getCollection(extension);
+        if (collection.doesExist(fileName))
         {
-            std::ifstream stream(dataFileCollections.getCollection(extension).getPath(fileName).c_str());
+            std::ifstream stream(collection.getPath(fileName));
 
             if (!stream.is_open())
                 throw std::runtime_error("failed to open translation file: " + fileName);

components/translation/translation.hpp

@@ -26,7 +26,7 @@ namespace Translation
     private:
         typedef std::map<std::string, std::string, std::less<>> ContainerType;
 
-        void loadData(ContainerType& container, const std::string& fileNameNoExtension, const std::string& extension,
+        void loadData(ContainerType& container, std::string_view fileNameNoExtension, std::string_view extension,
             const Files::Collections& dataFileCollections);
 
         void loadDataFromStream(ContainerType& container, std::istream& stream);