cleanup

Author: stuqdog

src/viam/sdk/module/service.cpp

@@ -195,7 +195,8 @@ struct ModuleService::ServiceImpl : viam::module::v1::ModuleService::Service {
         if (parent.parent_addr_ != new_parent_addr) {
             parent.parent_addr_ = std::move(new_parent_addr);
             Options opts{0, boost::none};
-            opts.set_check_every_interval(5).set_reconnect_every_interval(1);
+            opts.set_check_every_interval(std::chrono::seconds{5})
+                .set_reconnect_every_interval(std::chrono::seconds{1});
             parent.parent_ = RobotClient::at_local_socket(parent.parent_addr_, opts);
             parent.parent_->connect_logging();
         }

src/viam/sdk/robot/client.cpp

@@ -270,7 +270,7 @@ void RobotClient::check_connection() {
         }
         const auto* uri = viam_channel_.get_channel_addr();
         VIAM_SDK_LOG(error) << "Lost connection to machine at address " << uri << " with error "
-                            << what << ". Attempting to reconnect every " << reconnect_every
+                            << what << ". Attempting to reconnect every " << reconnect_every.count()
                             << "second(s)";
 
         viam_channel_.close();

src/viam/sdk/rpc/dial.cpp

@@ -223,7 +223,7 @@ std::chrono::seconds Options::reconnect_every_interval() const {
     return reconnect_every_interval_;
 }
 
-unsigned int Options::refresh_interval() const {
+std::chrono::seconds Options::refresh_interval() const {
     return refresh_interval_;
 }
 

src/viam/sdk/rpc/dial.hpp

@@ -119,7 +119,7 @@ class Options {
     Options(unsigned int refresh_interval, boost::optional<DialOptions> dial_options)
         : refresh_interval_(std::move(refresh_interval)), dial_options_(std::move(dial_options)) {}
 
-    unsigned int refresh_interval() const;
+    std::chrono::seconds refresh_interval() const;
     std::chrono::seconds check_every_interval() const;
     std::chrono::seconds reconnect_every_interval() const;