refactor: use correct header include order

include/sdbus-c++/ConvenienceApiClasses.h

@@ -27,16 +27,17 @@
 #ifndef SDBUS_CXX_CONVENIENCEAPICLASSES_H_
 #define SDBUS_CXX_CONVENIENCEAPICLASSES_H_
 
-#include <sdbus-c++/VTableItems.h>
 #include <sdbus-c++/Message.h>
 #include <sdbus-c++/TypeTraits.h>
 #include <sdbus-c++/Types.h>
+#include <sdbus-c++/VTableItems.h>
+
+#include <chrono>
+#include <cstdint>
+#include <future>
+#include <map>
 #include <string>
 #include <vector>
-#include <map>
-#include <chrono>
-#include <future>
-#include <cstdint>
 
 // Forward declarations
 namespace sdbus {

include/sdbus-c++/ConvenienceApiClasses.inl

@@ -27,18 +27,19 @@
 #ifndef SDBUS_CPP_CONVENIENCEAPICLASSES_INL_
 #define SDBUS_CPP_CONVENIENCEAPICLASSES_INL_
 
+#include <sdbus-c++/Error.h>
 #include <sdbus-c++/IObject.h>
 #include <sdbus-c++/IProxy.h>
 #include <sdbus-c++/Message.h>
 #include <sdbus-c++/MethodResult.h>
-#include <sdbus-c++/Types.h>
 #include <sdbus-c++/TypeTraits.h>
-#include <sdbus-c++/Error.h>
+#include <sdbus-c++/Types.h>
-#include <type_traits>
+
+#include <cassert>
+#include <exception>
 #include <string>
 #include <tuple>
-#include <exception>
+#include <type_traits>
-#include <cassert>
 
 namespace sdbus {
 

include/sdbus-c++/IConnection.h

@@ -28,11 +28,12 @@
 #define SDBUS_CXX_ICONNECTION_H_
 
 #include <sdbus-c++/TypeTraits.h>
-#include <string>
+
-#include <memory>
 #include <chrono>
 #include <cstdint>
+#include <memory>
 #include <optional>
+#include <string>
 
 struct sd_bus;
 struct sd_event;

include/sdbus-c++/IObject.h

@@ -27,13 +27,14 @@
 #ifndef SDBUS_CXX_IOBJECT_H_
 #define SDBUS_CXX_IOBJECT_H_
 
-#include <sdbus-c++/VTableItems.h>
 #include <sdbus-c++/ConvenienceApiClasses.h>
-#include <sdbus-c++/TypeTraits.h>
 #include <sdbus-c++/Flags.h>
+#include <sdbus-c++/TypeTraits.h>
+#include <sdbus-c++/VTableItems.h>
+
 #include <functional>
-#include <string>
 #include <memory>
+#include <string>
 #include <vector>
 
 // Forward declarations
@@ -385,7 +386,7 @@ namespace sdbus {
 
 }
 
-#include <sdbus-c++/VTableItems.inl>
 #include <sdbus-c++/ConvenienceApiClasses.inl>
+#include <sdbus-c++/VTableItems.inl>
 
 #endif /* SDBUS_CXX_IOBJECT_H_ */

include/sdbus-c++/IProxy.h

@@ -29,11 +29,12 @@
 
 #include <sdbus-c++/ConvenienceApiClasses.h>
 #include <sdbus-c++/TypeTraits.h>
-#include <string>
+
-#include <memory>
-#include <functional>
 #include <chrono>
+#include <functional>
 #include <future>
+#include <memory>
+#include <string>
 
 // Forward declarations
 namespace sdbus {

include/sdbus-c++/Message.h

@@ -27,24 +27,24 @@
 #ifndef SDBUS_CXX_MESSAGE_H_
 #define SDBUS_CXX_MESSAGE_H_
 
-#include <sdbus-c++/TypeTraits.h>
 #include <sdbus-c++/Error.h>
+#include <sdbus-c++/TypeTraits.h>
+
+#include <algorithm>
 #include <array>
-#include <string>
+#include <cassert>
-#include <utility>
+#include <cstdint>
-#include <vector>
+#include <functional>
+#include <map>
 #if __cplusplus >= 202002L
 #include <span>
 #endif
-#include <map>
+#include <string>
+#include <sys/types.h>
 #include <unordered_map>
 #include <utility>
-#include <cstdint>
-#include <cassert>
-#include <functional>
-#include <sys/types.h>
-#include <algorithm>
 #include <variant>
+#include <vector>
 
 // Forward declarations
 namespace sdbus {

include/sdbus-c++/MethodResult.h

@@ -28,9 +28,10 @@
 #define SDBUS_CXX_METHODRESULT_H_
 
 #include <sdbus-c++/Message.h>
+
 #include <cassert>
 
-// Forward declaration
+// Forward declarations
 namespace sdbus {
     class Error;
 }

include/sdbus-c++/TypeTraits.h

@@ -27,21 +27,20 @@
 #ifndef SDBUS_CXX_TYPETRAITS_H_
 #define SDBUS_CXX_TYPETRAITS_H_
 
-#include <type_traits>
-#include <string>
-#include <vector>
 #include <array>
-#include <variant>
+#include <cstdint>
+#include <functional>
+#include <map>
+#include <memory>
 #if __cplusplus >= 202002L
 #include <span>
 #endif
-#include <map>
+#include <string>
-#include <unordered_map>
-#include <cstdint>
-#include <functional>
-#include <memory>
 #include <tuple>
+#include <type_traits>
+#include <unordered_map>
 #include <variant>
+#include <vector>
 
 // Forward declarations
 namespace sdbus {

include/sdbus-c++/Types.h

@@ -29,11 +29,12 @@
 
 #include <sdbus-c++/Message.h>
 #include <sdbus-c++/TypeTraits.h>
+
+#include <memory>
 #include <string>
+#include <tuple>
 #include <type_traits>
 #include <typeinfo>
-#include <memory>
-#include <tuple>
 #include <utility>
 
 namespace sdbus {

include/sdbus-c++/VTableItems.h

@@ -26,11 +26,12 @@
 #ifndef SDBUS_CXX_VTABLEITEMS_H_
 #define SDBUS_CXX_VTABLEITEMS_H_
 
-#include <sdbus-c++/TypeTraits.h>
 #include <sdbus-c++/Flags.h>
+#include <sdbus-c++/TypeTraits.h>
+
 #include <string>
-#include <vector>
 #include <variant>
+#include <vector>
 
 namespace sdbus {
 

include/sdbus-c++/VTableItems.inl

@@ -26,10 +26,11 @@
 #ifndef SDBUS_CPP_VTABLEITEMS_INL_
 #define SDBUS_CPP_VTABLEITEMS_INL_
 
-#include <sdbus-c++/TypeTraits.h>
 #include <sdbus-c++/Error.h>
-#include <type_traits>
+#include <sdbus-c++/TypeTraits.h>
+
 #include <string>
+#include <type_traits>
 #include <vector>
 
 namespace sdbus {

src/Connection.cpp

@@ -25,20 +25,22 @@
  */
 
 #include "Connection.h"
-#include "SdBus.h"
+
+#include "sdbus-c++/Error.h"
+#include "sdbus-c++/Message.h"
+
 #include "MessageUtils.h"
-#include "Utils.h"
-#include <sdbus-c++/Message.h>
-#include <sdbus-c++/Error.h>
 #include "ScopeGuard.h"
+#include "SdBus.h"
+#include "Utils.h"
+
+#include <poll.h>
+#include <sys/eventfd.h>
 #include SDBUS_HEADER
 #ifndef SDBUS_basu // sd_event integration is not supported in basu-based sdbus-c++
 #include <systemd/sd-event.h>
 #endif
 #include <unistd.h>
-#include <poll.h>
-#include <sys/eventfd.h>
-#include <cstdint>
 
 namespace sdbus::internal {
 

src/Connection.h

@@ -27,15 +27,18 @@
 #ifndef SDBUS_CXX_INTERNAL_CONNECTION_H_
 #define SDBUS_CXX_INTERNAL_CONNECTION_H_
 
-#include <sdbus-c++/IConnection.h>
+#include "sdbus-c++/IConnection.h"
-#include <sdbus-c++/Message.h>
+
+#include "sdbus-c++/Message.h"
+
 #include "IConnection.h"
-#include "ScopeGuard.h"
 #include "ISdBus.h"
-#include SDBUS_HEADER
+#include "ScopeGuard.h"
+
 #include <memory>
-#include <thread>
 #include <string>
+#include SDBUS_HEADER
+#include <thread>
 #include <vector>
 
 struct sd_event_source;

src/Error.cpp

@@ -24,10 +24,12 @@
  * along with sdbus-c++. If not, see <http://www.gnu.org/licenses/>.
  */
 
-#include <sdbus-c++/Error.h>
+#include "sdbus-c++/Error.h"
-#include SDBUS_HEADER
+
 #include "ScopeGuard.h"
 
+#include SDBUS_HEADER
+
 namespace sdbus
 {
     sdbus::Error createError(int errNo, const std::string& customMsg)
@@ -40,4 +42,4 @@ namespace sdbus
         std::string message(customMsg + " (" + sdbusError.message + ")");
         return sdbus::Error(name, message);
     }
-}
+} // namespace sdbus

src/IConnection.h

@@ -27,12 +27,14 @@
 #ifndef SDBUS_CXX_INTERNAL_ICONNECTION_H_
 #define SDBUS_CXX_INTERNAL_ICONNECTION_H_
 
-#include <sdbus-c++/IConnection.h>
+#include "sdbus-c++/IConnection.h"
-#include <sdbus-c++/TypeTraits.h>
+
-#include SDBUS_HEADER
+#include "sdbus-c++/TypeTraits.h"
-#include <string>
+
-#include <memory>
 #include <functional>
+#include <memory>
+#include <string>
+#include SDBUS_HEADER
 #include <vector>
 
 // Forward declaration

src/Message.cpp

@@ -24,15 +24,18 @@
  * along with sdbus-c++. If not, see <http://www.gnu.org/licenses/>.
  */
 
-#include <sdbus-c++/Message.h>
+#include "sdbus-c++/Message.h"
-#include <sdbus-c++/Types.h>
+
-#include <sdbus-c++/Error.h>
+#include "sdbus-c++/Error.h"
-#include "MessageUtils.h"
+#include "sdbus-c++/Types.h"
-#include "ISdBus.h"
+
 #include "IConnection.h"
+#include "ISdBus.h"
+#include "MessageUtils.h"
 #include "ScopeGuard.h"
-#include SDBUS_HEADER
+
 #include <cassert>
+#include SDBUS_HEADER
 
 namespace sdbus {
 

src/Object.cpp

@@ -25,19 +25,21 @@
  */
 
 #include "Object.h"
-#include "MessageUtils.h"
+
-#include <sdbus-c++/IConnection.h>
+#include "sdbus-c++/Error.h"
-#include <sdbus-c++/Message.h>
+#include "sdbus-c++/Flags.h"
-#include <sdbus-c++/Error.h>
+#include "sdbus-c++/IConnection.h"
-#include <sdbus-c++/MethodResult.h>
+#include "sdbus-c++/Message.h"
-#include <sdbus-c++/Flags.h>
+
-#include "ScopeGuard.h"
 #include "IConnection.h"
+#include "MessageUtils.h"
+#include "ScopeGuard.h"
 #include "Utils.h"
 #include "VTableUtils.h"
+
+#include <cassert>
 #include SDBUS_HEADER
 #include <utility>
-#include <cassert>
 
 namespace sdbus::internal {
 

src/Object.h

@@ -27,15 +27,17 @@
 #ifndef SDBUS_CXX_INTERNAL_OBJECT_H_
 #define SDBUS_CXX_INTERNAL_OBJECT_H_
 
-#include <sdbus-c++/IObject.h>
+#include "sdbus-c++/IObject.h"
+
 #include "IConnection.h"
-#include SDBUS_HEADER
+
-#include <string>
-#include <list>
-#include <vector>
-#include <functional>
-#include <memory>
 #include <cassert>
+#include <functional>
+#include <list>
+#include <memory>
+#include <string>
+#include SDBUS_HEADER
+#include <vector>
 
 namespace sdbus::internal {
 

src/Proxy.cpp

@@ -25,16 +25,18 @@
  */
 
 #include "Proxy.h"
+
+#include "sdbus-c++/Error.h"
+#include "sdbus-c++/IConnection.h"
+#include "sdbus-c++/Message.h"
+
 #include "IConnection.h"
 #include "MessageUtils.h"
-#include "Utils.h"
-#include "sdbus-c++/Message.h"
-#include "sdbus-c++/IConnection.h"
-#include "sdbus-c++/Error.h"
 #include "ScopeGuard.h"
-#include SDBUS_HEADER
+#include "Utils.h"
+
 #include <cassert>
-#include <chrono>
+#include SDBUS_HEADER
 #include <utility>
 
 namespace sdbus::internal {

src/Proxy.h

@@ -27,14 +27,16 @@
 #ifndef SDBUS_CXX_INTERNAL_PROXY_H_
 #define SDBUS_CXX_INTERNAL_PROXY_H_
 
-#include <sdbus-c++/IProxy.h>
+#include "sdbus-c++/IProxy.h"
+
 #include "IConnection.h"
-#include SDBUS_HEADER
+
-#include <string>
-#include <memory>
 #include <deque>
-#include <vector>
+#include <memory>
 #include <mutex>
+#include <string>
+#include SDBUS_HEADER
+#include <vector>
 
 namespace sdbus::internal {
 

src/ScopeGuard.h

@@ -29,7 +29,6 @@
 
 #include <exception>
 #include <utility>
-#include <exception>
 
 // Straightforward, modern, easy-to-use RAII utility to perform work on scope exit in an exception-safe manner.
 //

src/Types.cpp

@@ -24,13 +24,15 @@
  * along with sdbus-c++. If not, see <http://www.gnu.org/licenses/>.
  */
 
-#include <sdbus-c++/Types.h>
+#include "sdbus-c++/Types.h"
-#include <sdbus-c++/Error.h>
+
+#include "sdbus-c++/Error.h"
+
 #include "MessageUtils.h"
-#include SDBUS_HEADER
+
-#include <cassert>
 #include <cerrno>
 #include <system_error>
+#include SDBUS_HEADER
 #include <unistd.h>
 
 namespace sdbus {
@@ -67,7 +69,7 @@ bool Variant::isEmpty() const
     return msg_.isEmpty();
 }
 
-void UnixFd::close()
+void UnixFd::close() // NOLINT(readability-make-member-function-const)
 {
     if (fd_ >= 0)
     {
@@ -82,7 +84,7 @@ int UnixFd::checkedDup(int fd)
         return fd;
     }
 
-    int ret = ::dup(fd);
+    int ret = ::dup(fd); // NOLINT(android-cloexec-dup) // TODO: verify
     if (ret < 0)
     {
         throw std::system_error(errno, std::generic_category(), "dup failed");
@@ -90,4 +92,4 @@ int UnixFd::checkedDup(int fd)
     return ret;
 }
 
-}
+} // namespace sdbus

src/VTableUtils.h

@@ -27,8 +27,8 @@
 #ifndef SDBUS_CXX_INTERNAL_VTABLEUTILS_H_
 #define SDBUS_CXX_INTERNAL_VTABLEUTILS_H_
 
-#include SDBUS_HEADER
 #include <stdbool.h>
+#include SDBUS_HEADER
 
 #ifdef __cplusplus
 extern "C" {