diff --git a/sdbusplus/message.hpp b/sdbusplus/message.hpp
index f7eecc8..2e15374 100644
--- a/sdbusplus/message.hpp
+++ b/sdbusplus/message.hpp
@@ -16,7 +16,7 @@
 namespace bus
 {
 struct bus;
-};
+}
 
 namespace message
 {
diff --git a/sdbusplus/message/append.hpp b/sdbusplus/message/append.hpp
index d57fc9b..655a9e1 100644
--- a/sdbusplus/message/append.hpp
+++ b/sdbusplus/message/append.hpp
@@ -22,7 +22,9 @@
  *  (This is an empty no-op function that is useful in some cases for
  *   variadic template reasons.)
  */
-inline void append(sdbusplus::SdBusInterface* intf, sd_bus_message* m){};
+inline void append(sdbusplus::SdBusInterface* /*intf*/, sd_bus_message* /*m*/)
+{
+}
 /** @brief Append data into an sdbus message.
  *
  *  @param[in] msg - The message to append to.
diff --git a/sdbusplus/message/read.hpp b/sdbusplus/message/read.hpp
index 4c52a71..736b56f 100644
--- a/sdbusplus/message/read.hpp
+++ b/sdbusplus/message/read.hpp
@@ -23,7 +23,9 @@
  *  (This is an empty no-op function that is useful in some cases for
  *   variadic template reasons.)
  */
-inline void read(sdbusplus::SdBusInterface* intf, sd_bus_message* m){};
+inline void read(sdbusplus::SdBusInterface* /*intf*/, sd_bus_message* /*m*/)
+{
+}
 /** @brief Read data from an sdbus message.
  *
  *  @param[in] msg - The message to read from.
diff --git a/sdbusplus/utility/type_traits.hpp b/sdbusplus/utility/type_traits.hpp
index 77d266b..2ab21c5 100644
--- a/sdbusplus/utility/type_traits.hpp
+++ b/sdbusplus/utility/type_traits.hpp
@@ -85,13 +85,13 @@
                                              std::index_sequence<Is...>)
 {
     return {(s[1 + Is])..., static_cast<char>(0)};
-};
+}
 
 template <std::size_t N>
 constexpr std::array<char, N - 2> strip_ends(const std::array<char, N>& s)
 {
     return strip_ends(s, std::make_index_sequence<N - 3>{});
-};
+}
 
 } // namespace utility