Always return 1 from request_completion_cb

No need to change the req_complete pointer as there are no sync objects
on requests that have a continuation attached.

Signed-off-by: Joseph Schuchart <schuchart@icl.utk.edu>

Author: devreal

ompi/mpiext/continue/c/continuation.c

@@ -700,15 +700,14 @@ static void handle_failed_cont(ompi_continuation_t *cont, int status, bool have_
 static int request_completion_cb(ompi_request_t *request)
 {
     assert(NULL != request->req_complete_cb_data);
-    int rc = 0;
     ompi_request_cont_data_t *req_cont_data;
 
     /* atomically swap the pointer here to avoid race with ompi_continue_global_wakeup */
     req_cont_data = (ompi_request_cont_data_t *)OPAL_THREAD_SWAP_PTR(&request->req_complete_cb_data, 0x0);
 
     if (NULL == req_cont_data) {
         /* the wakeup call took away our callback data */
-        return rc;
+        return 1;
     }
 
     ompi_continuation_t *cont = req_cont_data->cont_obj;
@@ -754,7 +753,7 @@ static int request_completion_cb(ompi_request_t *request)
 
     opal_free_list_return(&ompi_request_cont_data_freelist, &req_cont_data->super);
 
-    return rc;
+    return 1;
 }
 
 /* release all continuations, either by checking the requests for failure or just marking