Fix wait return code (#63)

pablogs9 · web-flow · commit da8d695e3083 · 2020-05-20T10:47:24.000+02:00
* Fix wait

* Remove run_session_status
diff --git a/rmw_microxrcedds_c/src/rmw_wait.c b/rmw_microxrcedds_c/src/rmw_wait.c
@@ -99,7 +99,7 @@ rmw_wait(
     session = &custom_subscription->owner_node->context->session;
   }
 
-  bool run_session_status = uxr_run_session_until_timeout(session, timeout);
+  uxr_run_session_until_timeout(session, timeout);
   bool buffered_status = false;
 
   // Check services
@@ -143,6 +143,5 @@ rmw_wait(
 
   EPROS_PRINT_TRACE()
     
-  return  (buffered_status) ? RMW_RET_OK : 
-          ((run_session_status) ? RMW_RET_ERROR : RMW_RET_TIMEOUT);
+  return  (buffered_status) ? RMW_RET_OK : RMW_RET_TIMEOUT;
 }

Original file line number	Diff line number	Diff line change
`@@ -99,7 +99,7 @@ rmw_wait(`
`99`	`99`	`session = &custom_subscription->owner_node->context->session;`
`100`	`100`	`}`
`101`	`101`
`102`		`- bool run_session_status = uxr_run_session_until_timeout(session, timeout);`
	`102`	`+ uxr_run_session_until_timeout(session, timeout);`
`103`	`103`	`bool buffered_status = false;`
`104`	`104`
`105`	`105`	`// Check services`
`@@ -143,6 +143,5 @@ rmw_wait(`
`143`	`143`
`144`	`144`	`EPROS_PRINT_TRACE()`
`145`	`145`
`146`		`- return (buffered_status) ? RMW_RET_OK :`
`147`		`- ((run_session_status) ? RMW_RET_ERROR : RMW_RET_TIMEOUT);`
	`146`	`+ return (buffered_status) ? RMW_RET_OK : RMW_RET_TIMEOUT;`
`148`	`147`	`}`