Do not use timeout.

Signed-off-by: Carlos Agüero <[email protected]>
gazebosim · Mar 12, 2024 · 687edcc · 687edcc
1 parent 0ed5942
commit 687edcc
Show file tree

Hide file tree

Showing 3 changed files with 4 additions and 4 deletions.
diff --git a/src/cmd/gz.cc b/src/cmd/gz.cc
@@ -251,7 +251,7 @@ extern "C" void cmdServiceReq(const char *_service,
   Node node;
   bool result;
 
-  if (_timeout == -1 || !strcmp(_repType, "gz.msgs.Empty"))
+  if (!strcmp(_repType, "gz.msgs.Empty"))
   {
     // One-way service.
     node.Request(_service, *req, 1000, *rep, result);

diff --git a/src/cmd/gz.hh b/src/cmd/gz.hh
@@ -51,9 +51,9 @@ extern "C" void cmdTopicPub(const char *_topic,
 /// \param[in] _service Service name.
 /// \param[in] _reqType Message type used in the request.
 /// \param[in] _repType Message type used in the response.
+///                     If "gz.msgs.Empty" is used, the request will be one-way
+///                     and _repType and _timeout will be ignored.
 /// \param[in] _timeout The request will timeout after '_timeout' ms.
-///                     If -1 is used, the request will be one-way and _repType
-///                     will be ignored.
 /// \param[in] _reqData Input data sent in the request.
 /// The format expected is the same used by Protobuf DebugString().
 /// E.g.: cmdServiceReq("/bar", "gz.msgs.StringMsg",

diff --git a/src/cmd/service_main.cc b/src/cmd/service_main.cc
@@ -73,7 +73,7 @@ void runServiceCommand(const ServiceOptions &_opt)
         // One-way service request.
         cmdServiceReq(_opt.service.c_str(),
             _opt.reqType.c_str(), "gz.msgs.Empty",
-            -1, _opt.reqData.c_str());
+            0, _opt.reqData.c_str());
       }
       else
       {