clang tidy

Author: stuqdog

src/viam/sdk/common/logger.cpp

@@ -125,9 +125,9 @@ void Logger::set_log_level(ll level) {
 }
 
 Logger::Logger(std::string name)
-    : name_(std::move(name)), level_(log_level::info), impl_(std::make_unique<impl>()) {};
+    : name_(std::move(name)), level_(log_level::info), impl_(std::make_unique<impl>()){};
 Logger::Logger(std::string name, log_level level)
-    : name_(std::move(name)), level_(level), impl_(std::make_unique<impl>(level)) {};
+    : name_(std::move(name)), level_(level), impl_(std::make_unique<impl>(level)){};
 
 logging::trivial::severity_level _log_level_to_severity_level(log_level level) {
     switch (level) {

src/viam/sdk/common/logger.hpp

@@ -124,4 +124,3 @@ void init_logging(std::ostream& custom_strm);
 #define VIAM_SDK_TRIVIAL_CUSTOM_FORMATTED_LOG(sev, msg) \
     ::viam::sdk::Logger::static_log(                    \
         ::viam::sdk::Logger::LogData<::viam::sdk::Logger::None>() << msg, sev, __FILE__, __LINE__)
-

src/viam/sdk/module/service.cpp

@@ -244,7 +244,7 @@ namespace {
 // moduleservice be aware of logging logic in a way that is confined to the implementation file.
 class custom_logging_buf : public std::stringbuf {
    public:
-    custom_logging_buf(std::shared_ptr<RobotClient> parent) : parent_(std::move(parent)) {};
+    custom_logging_buf(std::shared_ptr<RobotClient> parent) : parent_(std::move(parent)){};
     int sync() override {
         if (!parent_) {
             std::cerr << "Attempted to send custom gRPC log without parent address\n";