From 975e4f99084a36d7d3c636cb5520c326b425cf0c Mon Sep 17 00:00:00 2001 From: elsid Date: Sat, 25 Apr 2020 01:26:06 +0200 Subject: [PATCH 1/2] Fix warning -Wrange-loop-construct apps/openmw/mwgui/loadingscreen.cpp:81:36: warning: loop variable 'extension' of type 'const std::__cxx11::basic_string' creates a copy from type 'const std::__cxx11::basic_string' [-Wrange-loop-construct] for(auto const extension: supported_extensions) ^ apps/openmw/mwgui/loadingscreen.cpp:81:25: note: use reference type 'const std::__cxx11::basic_string &' to prevent copying for(auto const extension: supported_extensions) ^~~~~~~~~~~~~~~~~~~~~ & --- apps/openmw/mwgui/loadingscreen.cpp | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/apps/openmw/mwgui/loadingscreen.cpp b/apps/openmw/mwgui/loadingscreen.cpp index 54382ab4dc..7ab39915f8 100644 --- a/apps/openmw/mwgui/loadingscreen.cpp +++ b/apps/openmw/mwgui/loadingscreen.cpp @@ -78,7 +78,7 @@ namespace MWGui size_t pos = name.find_last_of('.'); if (pos != std::string::npos) { - for(auto const extension: supported_extensions) + for(auto const& extension: supported_extensions) { if (name.compare(pos, name.size() - pos, extension) == 0) { From ffacc30597ee38d6f4fb00154b1f325400e30c84 Mon Sep 17 00:00:00 2001 From: elsid Date: Sat, 25 Apr 2020 01:29:43 +0200 Subject: [PATCH 2/2] Use std::array for a list of extensions This list doesn't change and the size is known at compile time. --- apps/openmw/mwgui/loadingscreen.cpp | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/apps/openmw/mwgui/loadingscreen.cpp b/apps/openmw/mwgui/loadingscreen.cpp index 7ab39915f8..093a2f34cf 100644 --- a/apps/openmw/mwgui/loadingscreen.cpp +++ b/apps/openmw/mwgui/loadingscreen.cpp @@ -1,5 +1,7 @@ #include "loadingscreen.hpp" +#include + #include #include @@ -67,7 +69,7 @@ namespace MWGui mVFS->normalizeFilename(pattern); /* priority given to the left */ - std::list supported_extensions = {".tga", ".dds", ".ktx", ".png", ".bmp", ".jpeg", ".jpg"}; + const std::array supported_extensions {{".tga", ".dds", ".ktx", ".png", ".bmp", ".jpeg", ".jpg"}}; auto found = index.lower_bound(pattern); while (found != index.end())