Merge pull request #14756 from JoshuaVandaele/gcc16-warns-fix

Resolve various warnings

7 files changed, 14 insertions, 18 deletions

diff --git a/Source/Core/Common/CWDemangler.cpp b/Source/Core/Common/CWDemangler.cpp
index 8b4d700c2a..fd9503a111 100644
--- a/Source/Core/Common/CWDemangler.cpp
+++ b/Source/Core/Common/CWDemangler.cpp
@@ -111,7 +111,7 @@ static std::optional<std::size_t> find_split(std::string_view s, bool special,
   return std::nullopt;
 }
 
-ParseQualifiersResult parse_qualifiers(std::string_view str)
+static ParseQualifiersResult parse_qualifiers(std::string_view str)
 {
   std::string pre;
   std::string post;
@@ -171,7 +171,7 @@ ParseQualifiersResult parse_qualifiers(std::string_view str)
   return {pre, post, str};
 }
 
-std::optional<ParseDigitsResult> parse_digits(std::string_view str)
+static std::optional<ParseDigitsResult> parse_digits(std::string_view str)
 {
   if (str.empty())
     return std::nullopt;
diff --git a/Source/Core/Core/HW/HSP/HSP_DeviceGBPlayer.cpp b/Source/Core/Core/HW/HSP/HSP_DeviceGBPlayer.cpp
index ccc6e29e16..c7f37ca0b1 100644
--- a/Source/Core/Core/HW/HSP/HSP_DeviceGBPlayer.cpp
+++ b/Source/Core/Core/HW/HSP/HSP_DeviceGBPlayer.cpp
@@ -162,8 +162,8 @@ private:
   u8 m_bits_per_sample = 9;
 };
 
-CGBPlayer_mGBA::CGBPlayer_mGBA(Core::System& system, CHSPDevice_GBPlayer* player)
-    : IGBPlayer(system, player), m_gba_core{m_system, Config::GBPLAYER_GBA_INDEX}
+CGBPlayer_mGBA::CGBPlayer_mGBA(Core::System& system, CHSPDevice_GBPlayer* gbplayer)
+    : IGBPlayer(system, gbplayer), m_gba_core{m_system, Config::GBPLAYER_GBA_INDEX}
 {
   auto& core_timing = m_system.GetCoreTiming();
 
diff --git a/Source/Core/Core/IOS/FS/FileSystemCommon.cpp b/Source/Core/Core/IOS/FS/FileSystemCommon.cpp
index aa99f46da3..0e13192c5b 100644
--- a/Source/Core/Core/IOS/FS/FileSystemCommon.cpp
+++ b/Source/Core/Core/IOS/FS/FileSystemCommon.cpp
@@ -147,7 +147,7 @@ void FileSystem::DoStateRead(PointerWrap& p, const std::string& directory_path)
     return;
   }
 
-  Metadata metadata;
+  Metadata metadata{};
   p.Do(metadata.uid);
   p.Do(metadata.gid);
   p.Do(metadata.attribute);
diff --git a/Source/Core/Core/IOS/Network/Socket.h b/Source/Core/Core/IOS/Network/Socket.h
index 88a018dc9c..3617e244ce 100644
--- a/Source/Core/Core/IOS/Network/Socket.h
+++ b/Source/Core/Core/IOS/Network/Socket.h
@@ -17,23 +17,19 @@ typedef pollfd pollfd_t;
 #elif defined(__linux__) or defined(__APPLE__) or defined(__FreeBSD__) or defined(__NetBSD__) or   \
     defined(__OpenBSD__) or defined(__HAIKU__)
 #include <arpa/inet.h>
-#include <netdb.h>
-#include <sys/ioctl.h>
-#include <sys/socket.h>
-#include <sys/types.h>
-#if defined(ANDROID) || defined(__HAIKU__)
 #include <fcntl.h>
-#else
-#include <sys/fcntl.h>
-#endif
 #include <net/if.h>
+#include <netdb.h>
 #include <netinet/in.h>
 #include <poll.h>
+#include <sys/ioctl.h>
+#include <sys/socket.h>
+#include <sys/types.h>
 
 typedef struct pollfd pollfd_t;
 #else
+#include <fcntl.h>
 #include <netinet/in.h>
-#include <sys/fcntl.h>
 #include <sys/socket.h>
 #include <sys/types.h>
 #endif
diff --git a/Source/Core/DiscIO/DirectoryBlob.cpp b/Source/Core/DiscIO/DirectoryBlob.cpp
index d4f9e2bac7..47b9d73e41 100644
--- a/Source/Core/DiscIO/DirectoryBlob.cpp
+++ b/Source/Core/DiscIO/DirectoryBlob.cpp
@@ -895,8 +895,8 @@ DirectoryBlobPartition::DirectoryBlobPartition(
     const std::function<void(std::vector<FSTBuilderNode>* fst_nodes, FSTBuilderNode* dol_node)>&
         fst_callback,
     DirectoryBlobReader* blob)
-    : m_wrapped_partition(partition),
-      m_is_triforce(volume && volume->GetVolumeType() == Platform::Triforce)
+    : m_is_triforce(volume && volume->GetVolumeType() == Platform::Triforce),
+      m_wrapped_partition(partition)
 {
   std::vector<FSTBuilderNode> sys_nodes;
 
diff --git a/Source/Core/DolphinQt/Config/ARCodeWidget.cpp b/Source/Core/DolphinQt/Config/ARCodeWidget.cpp
index 02c9de4de0..12be8072ac 100644
--- a/Source/Core/DolphinQt/Config/ARCodeWidget.cpp
+++ b/Source/Core/DolphinQt/Config/ARCodeWidget.cpp
@@ -373,7 +373,7 @@ void ARCodeWidget::OnCodeToggleAllClicked()
     // file once per code.
     QSignalBlocker blocker(m_code_list);
 
-    for (int i = 0; i < m_ar_codes.size(); ++i)
+    for (int i = 0; i < static_cast<int>(m_ar_codes.size()); ++i)
     {
       m_ar_codes[i].enabled = new_state;
       m_code_list->item(i)->setCheckState(new_check_state);
diff --git a/Source/Core/DolphinQt/Config/GeckoCodeWidget.cpp b/Source/Core/DolphinQt/Config/GeckoCodeWidget.cpp
index ce00e7b53f..8a9da960bf 100644
--- a/Source/Core/DolphinQt/Config/GeckoCodeWidget.cpp
+++ b/Source/Core/DolphinQt/Config/GeckoCodeWidget.cpp
@@ -471,7 +471,7 @@ void GeckoCodeWidget::ToggleAllCodes()
     // file once per code.
     QSignalBlocker blocker(m_code_list);
 
-    for (int i = 0; i < m_gecko_codes.size(); ++i)
+    for (int i = 0; i < static_cast<int>(m_gecko_codes.size()); ++i)
     {
       m_gecko_codes[i].enabled = new_state;
       m_code_list->item(i)->setCheckState(new_check_state);