Switched to generic endian swap

Added missing <array> header

Addressed review comments

Restricted templated swaps to Packet

Made variable names more descriptive per review

Initial implementation of templated operators

 Changed to if constexpr to work around C4127 error

Conformed initialization style

Removed unused variable

examples/voip/Server.cpp

@@ -119,7 +119,7 @@ class NetworkAudioStream : public sf::SoundStream
                 break;
 
             // Extract the message ID
-            std::uint8_t id;
+            std::uint8_t id = 0;
             packet >> id;
 
             if (id == serverAudioData)

include/SFML/Network/Packet.hpp

@@ -29,17 +29,59 @@
 ////////////////////////////////////////////////////////////
 #include <SFML/Network/Export.hpp>
 
+#include <algorithm>
+#include <array>
 #include <cstddef>
+#include <cstring>
 #include <string>
 #include <vector>
 
-
 namespace sf
 {
 class String;
 class TcpSocket;
 class UdpSocket;
 
+namespace
+{
+////////////////////////////////////////////////////////////
+/// \brief Endian test
+///
+////////////////////////////////////////////////////////////
+inline bool isBigEndian()
+{
+    int one = 1;
+    return (*reinterpret_cast<char*>(&one)) != 1;
+};
+
+
+////////////////////////////////////////////////////////////
+/// \brief Generic endianness conversion; if host is big-endian,
+/// incoming network byte order can be handled without swap, and
+/// host byte order is already network byte order
+///
+////////////////////////////////////////////////////////////
+template <typename T>
+T swapEndian(T value)
+{
+    if (isBigEndian())
+    {
+        return value;
+    }
+
+    std::array<std::uint8_t, sizeof(T)> rawBytes;
+
+    auto* valueBytes = reinterpret_cast<std::uint8_t*>(&value);
+    std::memcpy(rawBytes.data(), valueBytes, sizeof(T));
+
+    std::array<std::uint8_t, sizeof(T)> swapped;
+    std::reverse_copy(rawBytes.begin(), rawBytes.end(), swapped.begin());
+    std::memcpy(valueBytes, swapped.data(), sizeof(T));
+
+    return *reinterpret_cast<T*>(valueBytes);
+}
+} // namespace
+
 ////////////////////////////////////////////////////////////
 /// \brief Utility class to build blocks of data to transfer
 ///        over the network
@@ -211,52 +253,21 @@ class SFML_NETWORK_API Packet
     ////////////////////////////////////////////////////////////
     /// \overload
     ////////////////////////////////////////////////////////////
-    Packet& operator>>(std::int8_t& data);
+    template <typename T>
+    Packet& operator>>(T& data)
+    {
+        if (checkSize(sizeof(data)))
+        {
+            std::memcpy(&data, &m_data[m_readPos], sizeof(data));
+            if constexpr (sizeof(T) > 1)
+            {
+                data = swapEndian(data);
+            }
+            m_readPos += sizeof(data);
+        }
 
-    ////////////////////////////////////////////////////////////
-    /// \overload
-    ////////////////////////////////////////////////////////////
-    Packet& operator>>(std::uint8_t& data);
-
-    ////////////////////////////////////////////////////////////
-    /// \overload
-    ////////////////////////////////////////////////////////////
-    Packet& operator>>(std::int16_t& data);
-
-    ////////////////////////////////////////////////////////////
-    /// \overload
-    ////////////////////////////////////////////////////////////
-    Packet& operator>>(std::uint16_t& data);
-
-    ////////////////////////////////////////////////////////////
-    /// \overload
-    ////////////////////////////////////////////////////////////
-    Packet& operator>>(std::int32_t& data);
-
-    ////////////////////////////////////////////////////////////
-    /// \overload
-    ////////////////////////////////////////////////////////////
-    Packet& operator>>(std::uint32_t& data);
-
-    ////////////////////////////////////////////////////////////
-    /// \overload
-    ////////////////////////////////////////////////////////////
-    Packet& operator>>(std::int64_t& data);
-
-    ////////////////////////////////////////////////////////////
-    /// \overload
-    ////////////////////////////////////////////////////////////
-    Packet& operator>>(std::uint64_t& data);
-
-    ////////////////////////////////////////////////////////////
-    /// \overload
-    ////////////////////////////////////////////////////////////
-    Packet& operator>>(float& data);
-
-    ////////////////////////////////////////////////////////////
-    /// \overload
-    ////////////////////////////////////////////////////////////
-    Packet& operator>>(double& data);
+        return *this;
+    }
 
     ////////////////////////////////////////////////////////////
     /// \overload
@@ -292,52 +303,17 @@ class SFML_NETWORK_API Packet
     ////////////////////////////////////////////////////////////
     /// \overload
     ////////////////////////////////////////////////////////////
-    Packet& operator<<(std::int8_t data);
-
-    ////////////////////////////////////////////////////////////
-    /// \overload
-    ////////////////////////////////////////////////////////////
-    Packet& operator<<(std::uint8_t data);
-
-    ////////////////////////////////////////////////////////////
-    /// \overload
-    ////////////////////////////////////////////////////////////
-    Packet& operator<<(std::int16_t data);
-
-    ////////////////////////////////////////////////////////////
-    /// \overload
-    ////////////////////////////////////////////////////////////
-    Packet& operator<<(std::uint16_t data);
-
-    ////////////////////////////////////////////////////////////
-    /// \overload
-    ////////////////////////////////////////////////////////////
-    Packet& operator<<(std::int32_t data);
-
-    ////////////////////////////////////////////////////////////
-    /// \overload
-    ////////////////////////////////////////////////////////////
-    Packet& operator<<(std::uint32_t data);
-
-    ////////////////////////////////////////////////////////////
-    /// \overload
-    ////////////////////////////////////////////////////////////
-    Packet& operator<<(std::int64_t data);
-
-    ////////////////////////////////////////////////////////////
-    /// \overload
-    ////////////////////////////////////////////////////////////
-    Packet& operator<<(std::uint64_t data);
-
-    ////////////////////////////////////////////////////////////
-    /// \overload
-    ////////////////////////////////////////////////////////////
-    Packet& operator<<(float data);
-
-    ////////////////////////////////////////////////////////////
-    /// \overload
-    ////////////////////////////////////////////////////////////
-    Packet& operator<<(double data);
+    template <typename T>
+    Packet& operator<<(T& data)
+    {
+        auto toWrite = data;
+        if constexpr (sizeof(T) > 1)
+        {
+            toWrite = swapEndian(data);
+        }
+        append(&toWrite, sizeof(toWrite));
+        return *this;
+    }
 
     ////////////////////////////////////////////////////////////
     /// \overload