Use plain timeval members instead of unique_ptr for X timeouts

The std::unique_ptr<timeval> indirection was carried over from
DashboardClientImplG5, where it composes with TCPSocket's existing
unique_ptr<timeval> member. In DashboardClientImplX it bought nothing:
no nullable semantics are needed because the constructor now installs
a 10 s default that every getter falls back to anyway.

Replace recv_timeout_ and send_timeout_ with value-typed timeval
members defaulted to {10, 0}. setReceiveTimeout / setSendTimeout
assign directly; the getters return the member. Behavior is
preserved (verified end-to-end against a stalling-server test that
times connect() at the configured value).

Author: Vinicius-T-Robotics

include/ur_client_library/ur/dashboard_client_implementation_x.h

@@ -186,11 +186,8 @@ class DashboardClientImplX : public DashboardClientImpl
 
   std::unique_ptr<httplib::Client> cli_;
   VersionInformation robot_api_version_;
-  // Caller-configured timeouts. Null until the corresponding setter is called explicitly;
-  // the getters fall back to the constructor default (10 s) in that case. Mirrors the
-  // recv_timeout_ pattern used by DashboardClientImplG5.
-  std::unique_ptr<timeval> recv_timeout_;
-  std::unique_ptr<timeval> send_timeout_;
+  timeval recv_timeout_ = {10, 0};
+  timeval send_timeout_ = {10, 0};
 };
 
 }  // namespace urcl

src/ur/dashboard_client_implementation_x.cpp

@@ -67,7 +67,7 @@ DashboardClientImplX::DashboardClientImplX(const std::string& host) : DashboardC
 
 void DashboardClientImplX::setReceiveTimeout(const timeval& timeout)
 {
-  recv_timeout_ = std::make_unique<timeval>(timeout);
+  recv_timeout_ = timeout;
   if (cli_)
   {
     cli_->set_read_timeout(std::chrono::seconds(timeout.tv_sec) + std::chrono::microseconds(timeout.tv_usec));
@@ -76,7 +76,7 @@ void DashboardClientImplX::setReceiveTimeout(const timeval& timeout)
 
 void DashboardClientImplX::setSendTimeout(const timeval& timeout)
 {
-  send_timeout_ = std::make_unique<timeval>(timeout);
+  send_timeout_ = timeout;
   if (cli_)
   {
     cli_->set_write_timeout(std::chrono::seconds(timeout.tv_sec) + std::chrono::microseconds(timeout.tv_usec));
@@ -160,33 +160,13 @@ timeval DashboardClientImplX::getConfiguredReceiveTimeout() const
   // return that. Otherwise fall back to the constructor default. See the constructor
   // comment for the rationale on the 10 s default (covers brake_release / program
   // load/upload/download without per-method timeouts).
-  timeval tv;
-  if (recv_timeout_ != nullptr)
-  {
-    tv = *recv_timeout_;
-  }
-  else
-  {
-    tv.tv_sec = 10;
-    tv.tv_usec = 0;
-  }
-  return tv;
+  return recv_timeout_;
 }
 
 timeval DashboardClientImplX::getConfiguredSendTimeout() const
 {
   // Mirrors getConfiguredReceiveTimeout. Default of 10 s matches the constructor.
-  timeval tv;
-  if (send_timeout_ != nullptr)
-  {
-    tv = *send_timeout_;
-  }
-  else
-  {
-    tv.tv_sec = 10;
-    tv.tv_usec = 0;
-  }
-  return tv;
+  return send_timeout_;
 }
 
 VersionInformation DashboardClientImplX::queryPolyScopeVersion()