diff --git a/include/sdbusplus/asio/connection.hpp b/include/sdbusplus/asio/connection.hpp
index 15edd1b..6f01596 100644
--- a/include/sdbusplus/asio/connection.hpp
+++ b/include/sdbusplus/asio/connection.hpp
@@ -133,7 +133,7 @@
         using FunctionTupleType =
             typename utility::decay_tuple<FunctionTuple>::type;
         constexpr bool returnWithMsg = []() {
-            if constexpr (std::tuple_size_v<FunctionTupleType> > 1)
+            if constexpr ((std::tuple_size_v<FunctionTupleType>) > 1)
             {
                 return std::is_same_v<
                     std::tuple_element_t<1, FunctionTupleType>,
diff --git a/include/sdbusplus/bus.hpp b/include/sdbusplus/bus.hpp
index 0fe6df8..a0b0b11 100644
--- a/include/sdbusplus/bus.hpp
+++ b/include/sdbusplus/bus.hpp
@@ -99,7 +99,7 @@
         ptrs.push_back(nullptr);
     }
 
-    explicit operator char* *()
+    explicit operator char**()
     {
         return const_cast<char**>(&ptrs.front());
     }
diff --git a/include/sdbusplus/message/native_types.hpp b/include/sdbusplus/message/native_types.hpp
index 6506b70..fe695c6 100644
--- a/include/sdbusplus/message/native_types.hpp
+++ b/include/sdbusplus/message/native_types.hpp
@@ -29,11 +29,11 @@
     string_wrapper(std::string&& str) : str(std::move(str))
     {}
 
-    operator const std::string &() const volatile&
+    operator const std::string&() const volatile&
     {
         return const_cast<const string_wrapper*>(this)->str;
     }
-    operator std::string &&() &&
+    operator std::string&&() &&
     {
         return std::move(str);
     }
@@ -94,11 +94,11 @@
     string_path_wrapper(std::string&& str) : str(std::move(str))
     {}
 
-    operator const std::string &() const volatile&
+    operator const std::string&() const volatile&
     {
         return const_cast<const string_path_wrapper*>(this)->str;
     }
-    operator std::string &&() &&
+    operator std::string&&() &&
     {
         return std::move(str);
     }