Skip to content

Commit ff669e7

Browse files
committed
code cleanup: clang is now a happier panda
Clang 5.1 on my mac was a sad panda compiling a couple of files, complaining about uninitialized stack variables. This commit makes clang a happier panda (or at least not so sad). Signed-off-by: Howard Pritchard <[email protected]>
1 parent c263461 commit ff669e7

File tree

2 files changed

+7
-5
lines changed

2 files changed

+7
-5
lines changed

ompi/mca/coll/base/coll_base_alltoall.c

Lines changed: 5 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -10,7 +10,7 @@
1010
* University of Stuttgart. All rights reserved.
1111
* Copyright (c) 2004-2005 The Regents of the University of California.
1212
* All rights reserved.
13-
* Copyright (c) 2013 Los Alamos National Security, LLC. All Rights
13+
* Copyright (c) 2013-2016 Los Alamos National Security, LLC. All Rights
1414
* reserved.
1515
* Copyright (c) 2014-2016 Research Organization for Information Science
1616
* and Technology (RIST). All rights reserved.
@@ -343,7 +343,8 @@ int ompi_coll_base_alltoall_intra_linear_sync(const void *sbuf, int scount,
343343
mca_coll_base_module_t *module,
344344
int max_outstanding_reqs)
345345
{
346-
int line, error, ri, si, rank, size, nreqs, nrreqs, nsreqs, total_reqs;
346+
int line, error, ri, si, rank, size, nrreqs, nsreqs, total_reqs;
347+
int nreqs = 0;
347348
char *psnd, *prcv;
348349
ptrdiff_t slb, sext, rlb, rext;
349350

@@ -565,7 +566,8 @@ int ompi_coll_base_alltoall_intra_basic_linear(const void *sbuf, int scount,
565566
struct ompi_communicator_t *comm,
566567
mca_coll_base_module_t *module)
567568
{
568-
int i, rank, size, err, nreqs, line;
569+
int i, rank, size, err, line;
570+
int nreqs = 0;
569571
char *psnd, *prcv;
570572
MPI_Aint lb, sndinc, rcvinc;
571573
ompi_request_t **req, **sreq, **rreq;

orte/runtime/orte_data_server.c

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -10,7 +10,7 @@
1010
* Copyright (c) 2004-2005 The Regents of the University of California.
1111
* All rights reserved.
1212
* Copyright (c) 2007 Cisco Systems, Inc. All rights reserved.
13-
* Copyright (c) 2012 Los Alamos National Security, LLC.
13+
* Copyright (c) 2012-2016 Los Alamos National Security, LLC.
1414
* All rights reserved
1515
* Copyright (c) 2015 Intel, Inc. All rights reserved.
1616
* $COPYRIGHT$
@@ -178,7 +178,7 @@ void orte_data_server(int status, orte_process_name_t* sender,
178178
uint32_t uid = UINT32_MAX;
179179
opal_pmix_data_range_t range;
180180
orte_data_req_t *req, *rqnext;
181-
orte_jobid_t jobid;
181+
orte_jobid_t jobid = ORTE_JOBID_INVALID;
182182

183183
OPAL_OUTPUT_VERBOSE((1, orte_debug_output,
184184
"%s data server got message from %s",

0 commit comments

Comments
 (0)