Fixed clangd warnings.

Author: neogeek

include/RhythmGameUtilities/Audio.hpp

@@ -1,5 +1,6 @@
 #pragma once
 
+#include <algorithm>
 #include <cmath>
 
 #ifdef _WIN32
@@ -23,10 +24,10 @@ extern "C"
      * @public
      */
 
-    PACKAGE_API int **ConvertSamplesToWaveform(float *samples, int size,
+    PACKAGE_API int **ConvertSamplesToWaveform(const float *samples, int size,
                                                int width, int height)
     {
-        auto waveform = new int *[width];
+        auto *waveform = new int *[width];
 
         auto step = floor(size / width);
         auto amp = height / 2;
@@ -40,19 +41,12 @@ extern "C"
 
             for (auto j = 0; j < step; j += 1)
             {
-                auto index = static_cast<int>(x * step + j);
+                auto index = static_cast<int>((x * step) + j);
 
                 auto datum = samples[index];
 
-                if (datum < min)
-                {
-                    min = datum;
-                }
-
-                if (datum > max)
-                {
-                    max = datum;
-                }
+                min = std::min(datum, min);
+                max = std::max(datum, max);
             }
 
             auto minY = static_cast<int>((1 + min) * amp);

include/RhythmGameUtilities/Common.hpp

@@ -26,7 +26,7 @@ extern "C"
 
     PACKAGE_API float Lerp(float a, float b, float t)
     {
-        return (1 - t) * a + b * t;
+        return ((1 - t) * a) + (b * t);
     }
 
     /**
@@ -82,7 +82,7 @@ inline std::vector<std::string> Split(const char *contents,
 }
 
 inline std::vector<std::string> FindAllMatches(const char *contents,
-                                               std::regex pattern)
+                                               const std::regex &pattern)
 {
     auto currentMatch =
         std::cregex_iterator(contents, contents + strlen(contents), pattern);
@@ -103,14 +103,14 @@ inline std::vector<std::string> FindAllMatches(const char *contents,
 }
 
 inline std::vector<std::string> FindMatchGroups(const char *contents,
-                                                std::regex pattern)
+                                                const std::regex &pattern)
 {
     auto currentMatch =
         std::cregex_iterator(contents, contents + strlen(contents), pattern);
 
     auto matches = std::vector<std::string>();
 
-    auto match = *currentMatch;
+    const auto &match = *currentMatch;
 
     for (auto i = 0; i < match.size(); i += 1)
     {

include/RhythmGameUtilities/File.hpp

@@ -22,7 +22,7 @@ inline std::vector<uint8_t> ReadBytesFromFile(const char *path)
 
     if (!file.is_open())
     {
-        std::cerr << "Failed to open " << path << "." << std::endl;
+        std::cerr << "Failed to open " << path << ".\n";
     }
 
     auto fileSize = file.tellg();
@@ -49,13 +49,13 @@ inline std::string ReadStringFromFile(const char *path)
 
     if (!file)
     {
-        std::cerr << "Failed to open " << path << "." << std::endl;
+        std::cerr << "Failed to open " << path << ".\n";
 
         return "";
     }
 
-    return std::string((std::istreambuf_iterator<char>(file)),
-                       std::istreambuf_iterator<char>());
+    return {(std::istreambuf_iterator<char>(file)),
+            std::istreambuf_iterator<char>()};
 }
 
 } // namespace RhythmGameUtilities

include/RhythmGameUtilities/Parsers.hpp

@@ -76,7 +76,7 @@ ParseSectionsFromChart(const char *contents)
                     std::regex_replace(values[k], std::regex("^\"|\"$"), "");
             }
 
-            items.push_back(std::make_pair(key, values));
+            items.emplace_back(key, values);
         }
 
         sections.insert({parts[1].c_str(), items});

include/RhythmGameUtilities/ParsersInternal.hpp

@@ -22,33 +22,33 @@ extern "C"
 
         *outSize = internalSections.size();
 
-        auto sections = (ChartSectionInternal *)malloc(
+        auto *sections = (ChartSectionInternal *)malloc(
             internalSections.size() * sizeof(ChartSectionInternal));
 
         int i = 0;
 
-        for (auto section = internalSections.begin();
-             section != internalSections.end(); section++)
+        for (auto &internalSection : internalSections)
         {
-            auto nameLength = section->first.size() + 1;
+            auto nameLength = internalSection.first.size() + 1;
             sections[i].name = (char *)malloc(nameLength);
-            strncpy(sections[i].name, section->first.c_str(), nameLength - 1);
+            strncpy(sections[i].name, internalSection.first.c_str(),
+                    nameLength - 1);
             sections[i].name[nameLength - 1] = '\0';
 
             sections[i].lines = (KeyValuePairInternal *)malloc(
-                section->second.size() * sizeof(KeyValuePairInternal));
+                internalSection.second.size() * sizeof(KeyValuePairInternal));
 
-            sections[i].lineCount = section->second.size();
+            sections[i].lineCount = internalSection.second.size();
 
-            for (auto j = 0; j < section->second.size(); j += 1)
+            for (auto j = 0; j < internalSection.second.size(); j += 1)
             {
-                auto keyLength = section->second[j].first.size() + 1;
+                auto keyLength = internalSection.second[j].first.size() + 1;
                 sections[i].lines[j].key = (char *)malloc(keyLength);
                 strncpy(sections[i].lines[j].key,
-                        section->second[j].first.c_str(), keyLength - 1);
+                        internalSection.second[j].first.c_str(), keyLength - 1);
                 sections[i].lines[j].key[keyLength - 1] = '\0';
 
-                auto values = section->second[j].second;
+                auto values = internalSection.second[j].second;
 
                 for (auto k = 0; k < values.size(); k += 1)
                 {

include/RhythmGameUtilities/Utilities.hpp

@@ -95,7 +95,7 @@ ConvertSecondsToTicks(float seconds, int resolution,
  */
 
 inline std::vector<std::tuple<int, int>>
-GenerateAdjacentKeyPairs(std::map<int, int> keyValuePairs)
+GenerateAdjacentKeyPairs(const std::map<int, int> &keyValuePairs)
 {
     auto adjacentKeyPairs = std::vector<std::tuple<int, int>>();
 
@@ -110,16 +110,15 @@ GenerateAdjacentKeyPairs(std::map<int, int> keyValuePairs)
 
     for (auto i = 0; i < sortedKeys.size() - 1; i += 1)
     {
-        adjacentKeyPairs.push_back(
-            std::make_tuple(sortedKeys[i], sortedKeys[i + 1]));
+        adjacentKeyPairs.emplace_back(sortedKeys[i], sortedKeys[i + 1]);
     }
 
     return adjacentKeyPairs;
 }
 
-inline std::vector<BeatBar> CalculateBeatBars(std::vector<Tempo> tempoChanges,
-                                              int resolution, int ts,
-                                              bool includeHalfNotes)
+inline std::vector<BeatBar>
+CalculateBeatBars(const std::vector<Tempo> &tempoChanges, int resolution,
+                  int ts, bool includeHalfNotes)
 {
     std::vector<BeatBar> beatBars;
 
@@ -146,7 +145,7 @@ inline std::vector<BeatBar> CalculateBeatBars(std::vector<Tempo> tempoChanges,
 
             if (includeHalfNotes && tick != endTick)
             {
-                beatBars.push_back({tick + resolution / 2, bpm});
+                beatBars.push_back({tick + (resolution / 2), bpm});
             }
         }
     }

include/RhythmGameUtilities/UtilitiesInternal.hpp

@@ -55,7 +55,7 @@ extern "C"
 
         *outSize = internalBeatBars.size();
 
-        auto beatBars =
+        auto *beatBars =
             (BeatBar *)malloc(internalBeatBars.size() * sizeof(BeatBar));
 
         for (auto i = 0; i < internalBeatBars.size(); i += 1)