Use in-class member initializers

See C++ Core Guidelines item C.45 for more information.

https://isocpp.github.io/CppCoreGuidelines/CppCoreGuidelines#c45-dont-define-a-default-constructor-that-only-initializes-data-members-use-in-class-member-initializers-instead

include/SFML/System/Angle.hpp

@@ -160,7 +160,7 @@ private:
     ////////////////////////////////////////////////////////////
     // Member data
     ////////////////////////////////////////////////////////////
-    float m_degrees; //!< Angle value stored as degrees
+    float m_degrees{0}; //!< Angle value stored as degrees
 };
 
 ////////////////////////////////////////////////////////////

include/SFML/System/Angle.inl

@@ -39,9 +39,7 @@ constexpr float positiveRemainder(float a, float b)
 
 
 ////////////////////////////////////////////////////////////
-constexpr Angle::Angle() : m_degrees(0.0f)
+constexpr Angle::Angle() = default;
-{
-}
 
 
 ////////////////////////////////////////////////////////////

include/SFML/System/Clock.hpp

@@ -140,7 +140,7 @@ private:
     ////////////////////////////////////////////////////////////
     // Member data
     ////////////////////////////////////////////////////////////
-    ClockImpl::time_point m_startTime; //!< Time of last reset
+    ClockImpl::time_point m_startTime{ClockImpl::now()}; //!< Time of last reset
 };
 
 } // namespace sf

include/SFML/System/MemoryInputStream.hpp

@@ -105,9 +105,9 @@ private:
     ////////////////////////////////////////////////////////////
     // Member data
     ////////////////////////////////////////////////////////////
-    const char*  m_data;   //!< Pointer to the data in memory
+    const char*  m_data{nullptr}; //!< Pointer to the data in memory
-    std::int64_t m_size;   //!< Total size of the data
+    std::int64_t m_size{0};       //!< Total size of the data
-    std::int64_t m_offset; //!< Current reading position
+    std::int64_t m_offset{0};     //!< Current reading position
 };
 
 } // namespace sf

include/SFML/System/String.hpp

@@ -56,7 +56,8 @@ public:
     ////////////////////////////////////////////////////////////
     // Static member data
     ////////////////////////////////////////////////////////////
-    static const std::size_t InvalidPos; //!< Represents an invalid position in the string
+    /// Represents an invalid position in the string
+    static inline const std::size_t InvalidPos{std::basic_string<std::uint32_t>::npos};
 
     ////////////////////////////////////////////////////////////
     /// \brief Default constructor

include/SFML/System/Time.hpp

@@ -119,7 +119,7 @@ private:
     ////////////////////////////////////////////////////////////
     // Member data
     ////////////////////////////////////////////////////////////
-    std::chrono::microseconds m_microseconds; //!< Time value stored as microseconds
+    std::chrono::microseconds m_microseconds{0}; //!< Time value stored as microseconds
 };
 
 ////////////////////////////////////////////////////////////

include/SFML/System/Time.inl

@@ -24,9 +24,7 @@
 
 
 ////////////////////////////////////////////////////////////
-constexpr Time::Time() : m_microseconds(0)
+constexpr Time::Time() = default;
-{
-}
 
 
 ////////////////////////////////////////////////////////////

include/SFML/System/Vector2.hpp

@@ -210,8 +210,8 @@ public:
     ////////////////////////////////////////////////////////////
     // Member data
     ////////////////////////////////////////////////////////////
-    T x; //!< X coordinate of the vector
+    T x{0}; //!< X coordinate of the vector
-    T y; //!< Y coordinate of the vector
+    T y{0}; //!< Y coordinate of the vector
 
 
     ////////////////////////////////////////////////////////////

include/SFML/System/Vector2.inl

@@ -25,9 +25,7 @@
 
 ////////////////////////////////////////////////////////////
 template <typename T>
-constexpr Vector2<T>::Vector2() : x(0), y(0)
+constexpr Vector2<T>::Vector2() = default;
-{
-}
 
 
 ////////////////////////////////////////////////////////////

include/SFML/System/Vector3.hpp

@@ -134,9 +134,9 @@ public:
     ////////////////////////////////////////////////////////////
     // Member data
     ////////////////////////////////////////////////////////////
-    T x; //!< X coordinate of the vector
+    T x{0}; //!< X coordinate of the vector
-    T y; //!< Y coordinate of the vector
+    T y{0}; //!< Y coordinate of the vector
-    T z; //!< Z coordinate of the vector
+    T z{0}; //!< Z coordinate of the vector
 };
 
 ////////////////////////////////////////////////////////////

include/SFML/System/Vector3.inl

@@ -25,9 +25,7 @@
 
 ////////////////////////////////////////////////////////////
 template <typename T>
-constexpr Vector3<T>::Vector3() : x(0), y(0), z(0)
+constexpr Vector3<T>::Vector3() = default;
-{
-}
 
 
 ////////////////////////////////////////////////////////////

src/SFML/System/Clock.cpp

@@ -32,9 +32,7 @@
 namespace sf
 {
 ////////////////////////////////////////////////////////////
-Clock::Clock() : m_startTime(ClockImpl::now())
+Clock::Clock() = default;
-{
-}
 
 
 ////////////////////////////////////////////////////////////

src/SFML/System/MemoryInputStream.cpp

@@ -33,9 +33,7 @@
 namespace sf
 {
 ////////////////////////////////////////////////////////////
-MemoryInputStream::MemoryInputStream() : m_data(nullptr), m_size(0), m_offset(0)
+MemoryInputStream::MemoryInputStream() = default;
-{
-}
 
 
 ////////////////////////////////////////////////////////////

src/SFML/System/String.cpp

@@ -35,13 +35,7 @@
 namespace sf
 {
 ////////////////////////////////////////////////////////////
-const std::size_t String::InvalidPos = std::basic_string<std::uint32_t>::npos;
+String::String() = default;
-
-
-////////////////////////////////////////////////////////////
-String::String()
-{
-}
 
 
 ////////////////////////////////////////////////////////////