additional PR feedback

Signed-off-by: Howard Pritchard <howardp@lanl.gov>
hpc · Jan 20, 2022 · 536bd20 · 536bd20
1 parent c192275
commit 536bd20
Show file tree

Hide file tree

Showing 3 changed files with 6 additions and 6 deletions.
diff --git a/ompi/mca/mtl/ofi/mtl_ofi.h b/ompi/mca/mtl/ofi/mtl_ofi.h
@@ -580,8 +580,8 @@ ompi_mtl_ofi_post_recv_excid_buffer(bool blocking, struct ompi_communicator_t *c
     ofi_req->convertor = NULL;
     ofi_req->req_started = false;
     ofi_req->status.MPI_ERROR = OMPI_SUCCESS;
-    ofi_req->remote_addr = NULL;
-    ofi_req->match_bits = NULL;
+    ofi_req->remote_addr = 0UL;
+    ofi_req->match_bits = 0UL;
     ofi_req->completion_count = 1;
     ofi_req->comm = comm;
 

diff --git a/opal/mca/btl/sm/btl_sm_fifo.h b/opal/mca/btl/sm/btl_sm_fifo.h
@@ -157,7 +157,7 @@ static inline bool sm_fifo_write_ep(mca_btl_sm_hdr_t *hdr, struct mca_btl_base_e
         opal_atomic_wmb();
         return mca_btl_sm_fbox_sendi(ep, 0xfe, &rhdr, sizeof(rhdr), NULL, 0);
     }
-    mca_btl_sm_try_fbox_setup (ep, hdr);
+    mca_btl_sm_try_fbox_setup(ep, hdr);
     hdr->next = SM_FIFO_FREE;
     sm_fifo_write(ep->fifo, rhdr);
 

diff --git a/opal/runtime/opal_finalize.c b/opal/runtime/opal_finalize.c
@@ -111,9 +111,9 @@ void opal_finalize_append_cleanup(opal_cleanup_fn_t cleanup_fn, const char *fn_n
     (void) fn_name;
 #endif
 
-    opal_mutex_lock (&opal_finalize_cleanup_fns_lock);
-    opal_list_append (&current_finalize_domain->super, &cleanup_item->super);
-    opal_mutex_unlock (&opal_finalize_cleanup_fns_lock);
+    opal_mutex_lock(&opal_finalize_cleanup_fns_lock);
+    opal_list_append(&current_finalize_domain->super, &cleanup_item->super);
+    opal_mutex_unlock(&opal_finalize_cleanup_fns_lock);
 }
 
 void opal_finalize_domain_init(opal_finalize_domain_t *domain, const char *domain_name)