Cleanup various sources of compiler warnings 78/4878/3
authorSam White <white67@illinois.edu>
Thu, 20 Dec 2018 22:07:23 +0000 (16:07 -0600)
committerSam White <white67@illinois.edu>
Fri, 21 Dec 2018 03:24:51 +0000 (21:24 -0600)
Change-Id: I3332663a0c0ac1f2df2af3cbc2ec3a9d30337163

31 files changed:
src/ck-core/ckarray.C
src/ck-core/ckarrayoptions.C
src/ck-core/ckcausalmlog.h
src/ck-core/cklocation.C
src/ck-core/ckmulticast.C
src/ck-ldb/LBAgent.C
src/ck-ldb/RecBipartLB.C
src/ck-ldb/RefinerTemp.h
src/ck-ldb/tm_bucket.c
src/ck-ldb/tm_mapping.c
src/ck-ldb/tm_tree.c
src/ck-perf/trace-projections.C
src/ck-perf/trace-projectionsBOC.h
src/ck-perf/trace-summary.C
src/ck-perf/trace-summaryBOC.h
src/ck-pics/picstreenode.h
src/conv-core/memory-charmdebug.c
src/conv-core/threads.c
src/conv-perf/traceCore.C
src/langs/bluegene/bigsim_logs.h
src/libs/ck-libs/ampi/ampi.C
src/libs/ck-libs/ampi/ampiimpl.h
src/libs/ck-libs/ampi/ddt.C
src/libs/ck-libs/ckloop/CkLoop.C
src/libs/ck-libs/ckloop/CkLoop.h
src/libs/ck-libs/collide/threadCollide.C
src/libs/ck-libs/io/ckio.C
src/libs/ck-libs/metis/GKlib/graph.c
src/libs/ck-libs/taskGraph/taskGraph.h
src/libs/conv-libs/master-slave/cms.c
src/util/pup_paged.h

index e042404c01a122253c2cacf40230fe6a9452ee20..64845aa41f3627c3243954c5c22c595a00a88b81 100644 (file)
@@ -833,9 +833,9 @@ CkArray::CkArray(CkArrayOptions &&opts,
                 CkMarshalledMessage &&initMsg)
   : locMgr(CProxy_CkLocMgr::ckLocalBranch(opts.getLocationManager())),
     locMgrID(opts.getLocationManager()),
+    mCastMgrID(opts.getMcastManager()),
     sectionAutoDelegate(opts.isSectionAutoDelegated()),
     initCallback(opts.getInitCallback()),
-    mCastMgrID(opts.getMcastManager()),
     thisProxy(thisgroup),
     stableLocations(opts.staticInsertion && !opts.anytimeMigration),
     numInitial(opts.getNumInitial()), isInserting(true), numPesInited(0)
index d2825634547c267096df97200e20ef516c374291..f94140d40598c63f9ad81b7aa3c063d1fa02bf70 100644 (file)
@@ -3,10 +3,10 @@
 #include "ckarrayoptions.h"
 
 CkArrayOptions::CkArrayOptions(void)  // Default: empty array
-    : numInitial(),
-      start(),
+    : start(),
       end(),
       step(),
+      numInitial(),
       bounds(),
       map(_defaultArrayMapID) {
   init();
index db872883fb56c9dd36480fd67634448b8beaca42..c0dc672bbbe62a4d56c5fd97d65e0e8683b9d74f 100644 (file)
@@ -221,7 +221,7 @@ public:
        /**
         * Default constructor.
         */ 
-       ChareMlogData():ticketTable(1000,0.3),snTable(100,0.4),teamTable(100,0.4){
+       ChareMlogData():teamTable(100,0.4),snTable(100,0.4),ticketTable(1000,0.3){
                tCount = 0;
                tProcessed = 0;
                numberHoles = 0;
index ba4cf3c3a7ecd142458c99888c5125652964204b..353ff6875c02dd6763be1c45aa2ef89ad71dc079 100644 (file)
@@ -1841,7 +1841,7 @@ represented array elements.
 CkLocRec::CkLocRec(CkLocMgr *mgr,bool fromMigration,
                    bool ignoreArrival, const CkArrayIndex &idx_, CmiUInt8 id_)
   :myLocMgr(mgr),idx(idx_), id(id_),
-        running(false),deletedMarker(NULL)
+        deletedMarker(NULL),running(false)
 {
 #if CMK_LBDB_ON
        DEBL((AA "Registering element %s with load balancer\n" AB,idx2str(idx)));
@@ -2110,8 +2110,8 @@ void CkLocMgr::callForAllRecords(CkLocFn fnPointer,CkArray *arr,void *data){
 
 /*************************** LocMgr: CREATION *****************************/
 CkLocMgr::CkLocMgr(CkArrayOptions opts)
-       :thisProxy(thisgroup),thislocalproxy(thisgroup,CkMyPe())
-       , idCounter(1)
+       :idCounter(1), thisProxy(thisgroup),
+  thislocalproxy(thisgroup,CkMyPe())
         , bounds(opts.getBounds())
 {
        DEBC((AA "Creating new location manager %d\n" AB,thisgroup));
@@ -3393,7 +3393,7 @@ CkLocRec *CkLocMgr::elementRec(const CkArrayIndex &idx) {
 #else
 //Include an out-of-bounds check if the element isn't found
   CmiUInt8 id;
-  CkLocRec *rec;
+  CkLocRec *rec = NULL;
   if (lookupID(idx, id) && (rec = elementNrec(id))) {
        return rec;
   } else {
index 8f7b6a75a70855d2f3199b951fc9b5fe15d8f435..f791b636ad2296d273521af155ff2ae91edb3452 100644 (file)
@@ -74,10 +74,10 @@ class reductionInfo {
         reductionMsgs futureMsgs;
 
     public:
-        reductionInfo(): storedCallback(NULL),
+        reductionInfo(): npProcessed(0),
+                         storedCallback(NULL),
                          storedClientParam(NULL),
-                         redNo(0),
-                         npProcessed(0) {
+                         redNo(0) {
             for (int8_t i=0; i<MAXFRAGS; i++)
                 lcount [i] = ccount [i] = gcount [i] = 0;
         }
@@ -167,10 +167,12 @@ class mCastEntry
         char flag;
        char grpSec;
     public:
-        mCastEntry(CkArrayID _aid): aid(_aid), numChild(0), asm_msg(NULL), asm_fill(0),
-                    oldc(NULL), newc(NULL), needRebuild(0), flag(COOKIE_NOTREADY), grpSec(0), localGrpElem(0) {}
-        mCastEntry(CkGroupID _gid): aid(_gid), numChild(0), asm_msg(NULL), asm_fill(0),
-                    oldc(NULL), newc(NULL), needRebuild(0), flag(COOKIE_NOTREADY), grpSec(1), localGrpElem(0) {}
+        mCastEntry(CkArrayID _aid): aid(_aid), numChild(0), localGrpElem(0), asm_msg(NULL),
+                   asm_fill(0), oldc(NULL), newc(NULL), needRebuild(0),
+                   flag(COOKIE_NOTREADY), grpSec(0) {}
+        mCastEntry(CkGroupID _gid): aid(_gid), numChild(0), localGrpElem(0), asm_msg(NULL),
+                   asm_fill(0), oldc(NULL), newc(NULL), needRebuild(0),
+                   flag(COOKIE_NOTREADY), grpSec(1) {}
         mCastEntry(mCastEntry *);
         /// Check if this tree is only a branch and has a parent
         inline int hasParent() { return parentGrp.get_val()?1:0; }
index 1658fd337988e25cdd0c5bf55c691ee80abf4215..c86c514c93532118e5aba56a738fdfbf9d01a015 100644 (file)
@@ -11,7 +11,7 @@
 
 #define PREF_RET_SIZE 5
 
-TopologyAgent::TopologyAgent(CentralLB::LDStats* lbDB,int p): stats(lbDB), Agent(p){
+TopologyAgent::TopologyAgent(CentralLB::LDStats* lbDB,int p): Agent(p), stats(lbDB){
        int i;
 
        LBtopoFn topofn;
index 0cffad2428ae928ebb2632e81f3aa08fcfe6c9de..4d97cffb40ab845f61d6da0449f62adab363fbc4 100644 (file)
@@ -535,7 +535,7 @@ void RefineBoundary(ObjGraph *ogr,vector<Vertex *> &partition1, vector<Vertex *>
 
 int modifypartitions(ObjGraph *ogr,vector<Vertex *> &partition1, vector<Vertex *> &partition2, BQueue *q1, BQueue *q2,int ec, int parent)
 {
-  int newedgecut;
+  int newedgecut = 0;
   if(q1->getBoundary()==1)//we are swapping vertex out of boundaryline1
   {
     int e2=vhelpers[q1->getVertextoswap()]->getEdgestopart2();
index 49e2a2dd3550be71df5a2a94c75b6bffe4fec661..0648e939e995541f1153158e2acf493dced0e783 100644 (file)
@@ -13,7 +13,7 @@
 class RefinerTemp {
 public:
   RefinerTemp(double _overload)
-    : overLoad(_overload), computes(0), P(CkNumPes())
+    : overLoad(_overload), P(CkNumPes()), computes(0)
     { 
     processors=0; 
     procFreq = new int[P];
index 115105bb7853ccb427edfea4478dfa39bcce6e00..3937f98ce430819d860e7a7237223367d687b302 100644 (file)
@@ -93,7 +93,7 @@ int bucket_id(int i,int j,bucket_list_t bucket_list){
 
 
 void  display_bucket(bucket_t *b){
-  printf("\tb.bucket=%p\n",b->bucket);
+  printf("\tb.bucket=%p\n",(void *)b->bucket);
   printf("\tb.bucket_len=%d\n",(int)b->bucket_len);
   printf("\tb.nb_elem=%d\n",(int)b->nb_elem);
 
index d22f6be24db8589f344bef15fd62db2a42d342e3..1d0ec57c6bf8561820bfd3ad9eca62d1822465f3 100644 (file)
@@ -689,7 +689,7 @@ void   build_synthetic_proc_id(tm_topology_t *topology){
 void update_comm_speed(double **comm_speed,int old_size,int new_size){
   double *old_tab,*new_tab;
   int i;
-  printf("comm speed [%p]: ",*comm_speed);
+  printf("comm speed [%p]: ", (void *)*comm_speed);
 
   old_tab=*comm_speed;
   new_tab=(double*)malloc(sizeof(double)*new_size);
index 81b30c2832342735bbfdcaaa69d342e18d890598..11c8bdc6f5bd3f6988f8b6beb11f8fd32e1f94b3 100644 (file)
@@ -73,7 +73,7 @@ void set_node(tree_t *node,tree_t ** child, int arity,tree_t *parent,int id,doub
 
 void display_node(tree_t *node){
   printf("child : %p\narity : %d\nparent : %p\nid : %d\nval : %f\nuniq : %d\n\n" ,
-        node->child,node->arity,node->parent,node->id,node->val,node->uniq);
+        (void *)node->child,node->arity,(void *)node->parent,node->id,node->val,node->uniq);
 }
 void clone_tree(tree_t *newptr,tree_t *oldptr){
   int i;
index 1e9bf66bbaa746e9612fb5132299c1b25a48ae94..acc900511c818158b03ab3cecaa6accb0a3179d3 100644 (file)
@@ -957,8 +957,8 @@ void LogEntry::pup(PUP::er &p)
          space2 = ' ';
           p | space2;
          if (p.isUnpacking()) {
-           userSuppliedNote = new char[length+1];
-           userSuppliedNote[length] = '\0';
+           userSuppliedNote = new char[length2+1];
+           userSuppliedNote[length2] = '\0';
          }
          PUParray(p,userSuppliedNote, length2);
          break;
@@ -1029,9 +1029,9 @@ void LogEntry::pup(PUP::er &p)
 }
 
 TraceProjections::TraceProjections(char **argv): 
-  curevent(0), inEntry(false), computationStarted(false),
-       converseExit(false), endTime(0.0), traceNestedEvents(false),
-       currentPhaseID(0), lastPhaseEvent(NULL), _logPool(NULL)
+  _logPool(NULL), curevent(0), inEntry(false), computationStarted(false),
+       traceNestedEvents(false), converseExit(false),
+       currentPhaseID(0), lastPhaseEvent(NULL), endTime(0.0)
 {
   //  CkPrintf("Trace projections dummy constructor called on %d\n",CkMyPe());
   if (CkpvAccess(traceOnPe) == 0) return;
index 9195e73f81523c05a562157ba6def955f4bc3e43..1462597979576dc47dd90dbfce32e5b93085d74b 100644 (file)
@@ -99,10 +99,9 @@ class KMeansBOC : public CBase_KMeansBOC {
  public:
  KMeansBOC(bool outlierAutomatic, int numKSeeds, int _peNumKeep,
           double _entryThreshold, bool outlierUsePhases) :
-  autoCompute(outlierAutomatic), numK(numKSeeds), 
-    peNumKeep(_peNumKeep), entryThreshold(_entryThreshold),
-    usePhases(outlierUsePhases) {};
- KMeansBOC(CkMigrateMessage *m):CBase_KMeansBOC(m) {};
+  autoCompute(outlierAutomatic), usePhases(outlierUsePhases),
+  numK(numKSeeds), peNumKeep(_peNumKeep), entryThreshold(_entryThreshold) {}
+ KMeansBOC(CkMigrateMessage *m):CBase_KMeansBOC(m) {}
   
   void startKMeansAnalysis();
   void flushCheck(bool someFlush);
index 78f9f271d20fe48930f0212a047cc6c0a2feec87..6c1285f211de1899c8fa3272dff9a799eff37f9d 100644 (file)
@@ -611,8 +611,8 @@ void BinEntry::write(FILE* fp)
   writeU(fp, getU());
 }
 
-TraceSummary::TraceSummary(char **argv):binStart(0.0),idleStart(0.0),
-                                       binTime(0.0),binIdle(0.0),msgNum(0)
+TraceSummary::TraceSummary(char **argv):msgNum(0),binStart(0.0),idleStart(0.0),
+                                       binTime(0.0),binIdle(0.0)
 {
   if (CkpvAccess(traceOnPe) == 0) return;
 
index 6defc4e7e1543d8516fc3bba53dbe075582547fd..f5b91b1d0b4a0e337c1fe9e23ceb1472017f20b0 100644 (file)
@@ -37,8 +37,8 @@ public:
 
 public:
   TraceSummaryBOC(void): count(0), bins(NULL), nBins(0), 
-    nTracedPEs(0), nextBinIndexCcs(0), firstTime(true) {};
-  TraceSummaryBOC(CkMigrateMessage *m):CBase_TraceSummaryBOC(m) {};
+    nTracedPEs(0), firstTime(true), nextBinIndexCcs(0) {}
+  TraceSummaryBOC(CkMigrateMessage *m):CBase_TraceSummaryBOC(m) {}
   void startSumOnly();
   void askSummary(int size);
   void sendSummaryBOC(double *results, int n);
index 4f4a21f59e116b8ab159cb225eb655de97e7b4e7..87b2df96f1932936faa8f1d914261b13bb35900b 100644 (file)
@@ -37,7 +37,7 @@ public:
 
   Condition(std::string n, int _varIndex, Operator _op, int _baseIndex,
       double _threshold, CompareSymbol c) : varIndex(_varIndex),
-      baseIndex(_baseIndex), threshold(_threshold), thresholdIndex(-1),
+      baseIndex(_baseIndex), thresholdIndex(-1), threshold(_threshold),
       symbol(c), op(_op) {
       name = n;
   }
index 9f844472c0b8366ebce1d203a907109cac783937..8950cb8bfcea38fb09d848b89d984cbd44c7cc98 100644 (file)
@@ -168,7 +168,7 @@ int Slot_StackTrace(void *s, void ***stack) {
 
 static void printSlot(Slot *s) {
   CmiPrintf("[%d] Leaked block of %d bytes at %p:\n",
-           CmiMyPe(), s->userSize, SlotToUser(s));
+           CmiMyPe(), s->userSize, (void *)SlotToUser(s));
   CmiBacktracePrint(s->from,s->stackLen);
 }
 
@@ -236,7 +236,7 @@ void cpd_memory_single_pup(Slot* list, pup_er p) {
       pup_pointers(p, cur->from, cur->stackLen);
     else {
       void *myNULL = NULL;
-      printf("Block %p has no stack!\n",cur);
+      printf("Block %p has no stack!\n",(void *)cur);
       pup_pointer(p, &myNULL);
     }
     }
@@ -421,7 +421,7 @@ void check_memory_leaks(LeakSearchInfo *info) {
     begin_bss = ntohl(((char**)(req->extra))[2]);
     end_bss = ntohl(((char**)(req->extra))[3]) - sizeof(char*) + 1;
   }*/
-  printf("scanning stack from %p to %p\n", begin_stack, end_stack);
+  printf("scanning stack from %p to %p\n", (void *)begin_stack, (void *)end_stack);
   for (scanner = begin_stack; scanner < end_stack; scanner+=sizeof(char*)) {
     fnd = (Slot**)CkHashtableGet(table, scanner);
     //if (fnd != NULL) printf("scanning stack %p, %d\n",*fnd,isLeakSlot(*fnd));
@@ -434,7 +434,7 @@ void check_memory_leaks(LeakSearchInfo *info) {
       PCQueuePush(inProgress, (char*)found);
     }
   }
-  printf("scanning data from %p to %p\n", info->begin_data, info->end_data);
+  printf("scanning data from %p to %p\n", (void *)info->begin_data, (void *)info->end_data);
   for (scanner = info->begin_data; scanner < info->end_data; scanner+=sizeof(char*)) {
     //fprintf(fd, "scanner = %p\n",scanner);
     //fflush(fd);
@@ -449,7 +449,7 @@ void check_memory_leaks(LeakSearchInfo *info) {
       PCQueuePush(inProgress, (char*)found);
     }
   }
-  printf("scanning bss from %p to %p\n", info->begin_bss, info->end_bss);
+  printf("scanning bss from %p to %p\n", (void *)info->begin_bss, (void *)info->end_bss);
   for (scanner = info->begin_bss; scanner < info->end_bss; scanner+=sizeof(char*)) {
     //printf("bss: %p %p\n",scanner,*(char**)scanner);
     fnd = (Slot**)CkHashtableGet(table, scanner);
@@ -611,7 +611,7 @@ void printAllocationTree(AllocationPoint *node, FILE *fd, int depth) {
   if (node==NULL) return;
   for (child = node->firstChild; child != NULL; child = child->sibling) numChildren ++;
   for (i=0; i<depth; ++i) fprintf(fd, " ");
-  fprintf(fd, "node %p: bytes=%d, count=%d, child=%d\n",node->key,node->size,node->count,numChildren);
+  fprintf(fd, "node %p: bytes=%d, count=%d, child=%d\n",(void *)node->key,node->size,node->count,numChildren);
   printAllocationTree(node->sibling, fd, depth);
   printAllocationTree(node->firstChild, fd, depth+2);
 }
@@ -698,17 +698,17 @@ AllocationPoint * CreateAllocationTree(int *nodesCount) {
   fprintf(fd, "digraph %s {\n", filename);
   it = CkHashtableGetIterator(table);
   while ((startscan=(AllocationPoint**)CkHashtableIteratorNext(it,NULL))!=NULL) {
-    fprintf(fd, "\t\"n%p\" [label=\"%p\\nsize=%d\\ncount=%d\"];\n",*startscan,(*startscan)->key,
+    fprintf(fd, "\t\"n%p\" [label=\"%p\\nsize=%d\\ncount=%d\"];\n",(void *)*startscan,(void *)(*startscan)->key,
           (*startscan)->size,(*startscan)->count);
     for (scan = (*startscan)->next; scan != *startscan; scan = scan->next) {
-      fprintf(fd, "\t\"n%p\" [label=\"%p\\nsize=%d\\ncount=%d\"];\n",scan,scan->key,scan->size,scan->count);
+      fprintf(fd, "\t\"n%p\" [label=\"%p\\nsize=%d\\ncount=%d\"];\n",(void *)scan,(void *)scan->key,scan->size,scan->count);
     }
   }
   CkHashtableIteratorSeekStart(it);
   while ((startscan=(AllocationPoint**)CkHashtableIteratorNext(it,NULL))!=NULL) {
-    fprintf(fd, "\t\"n%p\" -> \"n%p\";\n",(*startscan)->parent,(*startscan));
+    fprintf(fd, "\t\"n%p\" -> \"n%p\";\n",(void *)(*startscan)->parent,(void *)(*startscan));
     for (scan = (*startscan)->next; scan != *startscan; scan = scan->next) {
-      fprintf(fd, "\t\"n%p\" -> \"n%p\";\n",scan->parent,scan);
+      fprintf(fd, "\t\"n%p\" -> \"n%p\";\n",(void *)scan->parent,(void *)scan);
     }
   }
   fprintf(fd, "}\n");
@@ -956,10 +956,10 @@ static void CheckAllCRC(void) {
     crc1 = crc32_update((unsigned char*)cur->from, cur->stackLen*sizeof(void*), crc1);
     crc2 = crc32_initial((unsigned char*)SlotToUser(cur), cur->userSize);
     /* Here we can check if a modification has occured */
-    if (reportMEM && cur->slotCRC != crc1) CmiPrintf("CRC: Object %d modified slot for %p\n",memory_chare_id,SlotToUser(cur));
+    if (reportMEM && cur->slotCRC != crc1) CmiPrintf("CRC: Object %d modified slot for %p\n",memory_chare_id,(void *)SlotToUser(cur));
     cur->slotCRC = crc1;
     if (reportMEM && cur->userCRC != crc2 && memory_chare_id != cur->chareID)
-      CmiPrintf("CRC: Object %d modified memory of object %d for %p\n",memory_chare_id,cur->chareID,SlotToUser(cur));
+      CmiPrintf("CRC: Object %d modified memory of object %d for %p\n",memory_chare_id,cur->chareID,(void *)SlotToUser(cur));
     cur->userCRC = crc2;
   SLOT_ITERATE_END
 }
@@ -1031,7 +1031,7 @@ static void checkBackup(void) {
       int res = memcmp(ptr+sizeof(Slot), ((char*)cur)+sizeof(Slot), cur->userSize + cur->stackLen*sizeof(void*));
       if (res != 0) {
         cur->magic |= MODIFIED;
-        if (reportMEM) CmiPrintf("CPD: Object %d modified memory of object %d for %p on pe %d\n",memory_chare_id,cur->chareID,cur+1,CmiMyPe());
+        if (reportMEM) CmiPrintf("CPD: Object %d modified memory of object %d for %p on pe %d\n",memory_chare_id,cur->chareID,(void *)(cur+1),CmiMyPe());
       }
     }
 
@@ -1145,9 +1145,9 @@ void CpdCheckMemory(void) {
   if (CpdMemBackup) checkBackup();
   SLOT_ITERATE_START(cur)
     if (cur->magic == SLOTMAGIC_FREED) CmiAbort("SLOT deallocate in list");
-    if (cur->from == NULL) printf("SLOT %p has no stack\n",cur);
+    if (cur->from == NULL) printf("SLOT %p has no stack\n",(void *)cur);
 #ifndef CMK_SEPARATE_SLOT
-    if (cur->next == NULL) printf("SLOT %p has null next!\n",cur);
+    if (cur->next == NULL) printf("SLOT %p has null next!\n",(void *)cur);
 #endif
   SLOT_ITERATE_END
 }
@@ -1469,7 +1469,7 @@ static void meta_init(char **argv) {
 }
 
 static void *meta_malloc(size_t size) {
-  void *user;
+  void *user = NULL;
   if (memory_charmdebug_internal==0) {
     Slot *s;
     dumpStackFrames();
@@ -1489,7 +1489,7 @@ static void *meta_malloc(size_t size) {
     }
     if (disableVerbosity == 0) {
       disableVerbosity = 1;
-      CmiPrintf("allocating %p: %d bytes\n",s,size);
+      CmiPrintf("allocating %p: %d bytes\n",(void *)s,size);
       disableVerbosity = 0;
     }
   } else {
@@ -1543,7 +1543,7 @@ static void meta_free(void *mem) {
 
     if (disableVerbosity == 0) {
       disableVerbosity = 1;
-      CmiPrintf("freeing %p\n",mem);
+      CmiPrintf("freeing %p\n",(void *)mem);
       disableVerbosity = 0;
     }
 
index 440910901a0dbbba5f97e5fef01016cdb6123457..9a67b562b66040dce733e82ccf7f374907ccd762 100644 (file)
@@ -298,7 +298,7 @@ void CthAliasEnable(CthThreadBase *t) {
   if (!t->isMigratable) return;
   if (t==_curMappedStack) return; /* don't re-map */
   _curMappedStack=t;
-  if (0) printf("Mmapping in thread %p from runtime stack %p\n",t,&s);
+  CthDebug("Mmapping in thread %p from runtime stack %p\n", (void*)t, (void*)&s);
 
   /* Linux mmap flag MAP_POPULATE, to pre-fault in all the pages,
      only seems to slow down overall performance. */
index 8b7f3606aca04402ebfd09ad19bb452bf5cbd47c..34723a648d566f2816b1be628ac6e17d14c0da27 100644 (file)
@@ -304,7 +304,7 @@ void TraceEntry::write(FILE* fp, int prevLID, int prevSeek, int nextLID, int nex
 
 /***************** Class TraceLogger Definition *****************/
 TraceLogger::TraceLogger(char* program, int b):
-       numLangs(1), numEntries(0), lastWriteFlag(false), prevLID(0), prevSeek(0)
+       numEntries(0), numLangs(1), lastWriteFlag(false), prevLID(0), prevSeek(0)
 {
   binary = b;
 
index 62e9f0bfcdf065925371865e1540ff610196e7c8..3a4ea6266fe8ace1f422914751aa5abeab677cf6 100644 (file)
@@ -88,7 +88,7 @@ public:
   char   eType;
   bgEvents(): index(-1) {}
   bgEvents(void *d, int idx, double t, bgEventCallBackFn fn, void *ptr, char e):
-       data(d), index(idx), rTime(t), callbackFn(fn), usrPtr(ptr), eType(e) {}
+       callbackFn(fn), usrPtr(ptr), data(d), index(idx), rTime(t), eType(e) {}
   inline void update(double startT, double recvT, int e) {
        if (eType==e) callbackFn(data, startT+rTime, recvT, usrPtr);
   }
@@ -247,8 +247,8 @@ public:
   CkQ<BgTimeLog *>   sendingLogs;      // send buffered
 #endif
 public:
-  BgTimeLineRec(): timeline(1024), commit(0), counter(1), correctSendIdx(0), 
-                  startIdx(0), bgCurLog(NULL), bgPrevLog(NULL) {
+  BgTimeLineRec(): timeline(1024), commit(0), startIdx(0), correctSendIdx(0),
+                  counter(1), bgCurLog(NULL), bgPrevLog(NULL) {
       if (bgcorroff) startCorrFlag=0; else startCorrFlag=1;
       minCorrection = INVALIDTIME;
     }
index a3629dddc30cac42080a20f9366a96563b522a88..b6527af675b4dc8fad545451cd494f75047ca386 100644 (file)
@@ -1231,8 +1231,8 @@ class ampiWorlds : public CBase_ampiWorlds {
 
 //-------------------- ampiParent -------------------------
 ampiParent::ampiParent(MPI_Comm worldNo_,CProxy_TCharm threads_,int nRanks_) noexcept
-  : threads(threads_), worldNo(worldNo_), isTmpRProxySet(false), ampiReqs(64, &reqPool),
-    myDDT(ampiPredefinedTypes), predefinedOps(ampiPredefinedOps)
+  : threads(threads_), ampiReqs(64, &reqPool), myDDT(ampiPredefinedTypes),
+    worldNo(worldNo_), predefinedOps(ampiPredefinedOps), isTmpRProxySet(false)
 {
   int barrier = 0x1234;
   STARTUP_DEBUG("ampiParent> starting up")
index 880f434c0d19a5e7ddd9f2719cf88665ec54ee4b..a870eba403c18aae9d02e93177903d53b6cda3d3 100644 (file)
@@ -717,7 +717,7 @@ class ampiCommStruct {
     : size(-1), commType(INTRA), ampiTopo(NULL), topoType(MPI_UNDEFINED)
   {}
   ampiCommStruct(MPI_Comm comm_,const CkArrayID &id_,int size_) noexcept
-    : comm(comm_), ampiID(id_),size(size_), indices(size_), commType(WORLD),
+    : comm(comm_), ampiID(id_),size(size_), commType(WORLD), indices(size_),
       ampiTopo(NULL), topoType(MPI_UNDEFINED)
   {}
   ampiCommStruct(MPI_Comm comm_,const CkArrayID &id_, const vector<int> &indices_) noexcept
index aacd2d0d311bffdacbe0675fb114406bf737ef78..96efa75a50dedb359c086eeed75708cab8848ca8 100644 (file)
@@ -503,15 +503,14 @@ CkDDT_DataType::CkDDT_DataType(int type) noexcept : datatype(type)
            type, size, extent, iscontig);
 }
 
-
 CkDDT_DataType::CkDDT_DataType(int datatype, int size, MPI_Aint extent, int count, MPI_Aint lb,
                                MPI_Aint ub, bool iscontig, int baseSize, MPI_Aint baseExtent,
                                CkDDT_DataType* baseType, int numElements, int baseIndex,
                                MPI_Aint trueExtent, MPI_Aint trueLB) noexcept :
-    datatype(datatype), refCount(1), size(size), extent(extent), count(count), lb(lb), ub(ub),
-    trueExtent(trueExtent), trueLB(trueLB), iscontig(iscontig), baseSize(baseSize),
-    baseExtent(baseExtent), baseType(baseType), numElements(numElements), baseIndex(baseIndex),
-    isAbsolute(false)
+    iscontig(iscontig), isAbsolute(false), size(size), count(count), datatype(datatype),
+    refCount(1), baseSize(baseSize), baseIndex(baseIndex), numElements(numElements),
+    extent(extent), ub(ub), lb(lb), trueExtent(trueExtent), trueLB(trueLB),
+    baseExtent(baseExtent), baseType(baseType)
 {
   if (baseType) {
     baseType->incRefCount();
@@ -519,22 +518,22 @@ CkDDT_DataType::CkDDT_DataType(int datatype, int size, MPI_Aint extent, int coun
 }
 
 CkDDT_DataType::CkDDT_DataType(const CkDDT_DataType &obj, MPI_Aint _lb/*=0*/, MPI_Aint _extent/*=0*/) noexcept :
-  datatype(obj.datatype)
-  ,refCount(1)
+  iscontig(obj.iscontig)
+  ,isAbsolute(obj.isAbsolute)
   ,size(obj.size)
-  ,extent(obj.extent)
   ,count(obj.count)
-  ,lb(obj.lb)
+  ,datatype(obj.datatype)
+  ,refCount(1)
+  ,baseSize(obj.baseSize)
+  ,baseIndex(obj.baseIndex)
+  ,numElements(obj.numElements)
+  ,extent(obj.extent)
   ,ub(obj.ub)
+  ,lb(obj.lb)
   ,trueExtent(obj.trueExtent)
   ,trueLB(obj.trueLB)
-  ,iscontig(obj.iscontig)
-  ,baseSize(obj.baseSize)
   ,baseExtent(obj.baseExtent)
   ,baseType(obj.baseType)
-  ,numElements(obj.numElements)
-  ,baseIndex(obj.baseIndex)
-  ,isAbsolute(obj.isAbsolute)
   ,name(obj.name)
 {
   if (baseType) {
index aac5af6054e6d6063a214af38d46ed8034611a63..bd0e0a868feb209025fcbd034be6ee0c946f4217 100644 (file)
@@ -808,7 +808,7 @@ void CurLoopInfo::doWorkForMyPe() {
   int lastDynamic = lowerIndex + (myRank+1)*(upperIndex - lowerIndex)/numHelpers;
   if(lastDynamic > upperIndex) lastDynamic = upperIndex; // for the last PE.
 
-  int i, j;
+  int i, j = 0;
 
   // TODO: make numChunks smaller as needed.
 
index b4c88d403aa719faf0896a40e00d91c1658463be..2599d3325f44f989cca76816a349d18f8a780a98 100644 (file)
@@ -59,8 +59,8 @@ private:
     std::atomic<int> numDynamicChunksFired{0};
 
 public:
-    CurLoopInfo(int maxChunks):numChunks(0),fnPtr(NULL), lowerIndex(-1), upperIndex(0),
-            paramNum(0), param(NULL), curChunkIdx(-1), finishFlag(0), redBufs(NULL), bufSpace(NULL), inited(0) {
+    CurLoopInfo(int maxChunks):curChunkIdx(-1),numChunks(0),fnPtr(NULL), lowerIndex(-1), upperIndex(0),
+            paramNum(0), param(NULL), redBufs(NULL), bufSpace(NULL), finishFlag(0), inited(0) {
         redBufs = new void *[maxChunks];
         bufSpace = new char[maxChunks * CMI_CACHE_LINE_SIZE];
         for (int i=0; i<maxChunks; i++) redBufs[i] = (void *)(bufSpace+i*CMI_CACHE_LINE_SIZE);
index e0b4d6fbd733c668e17757f3faded433d2ee6ad8..e975f00a7142c7b56627871d3616fc0209b8a4ce 100644 (file)
@@ -110,7 +110,7 @@ public:
        threadCollide(const CProxy_TCharm &threads,
                const CProxy_threadCollideMgr &mgr_,
                const CollideHandle &collide_) 
-               :super(threads), mgr(mgr_), collide(collide_)
+               :super(threads), collide(collide_), mgr(mgr_)
        {
                arriving();
                /// Wake up the blocked thread in COLLIDE_Init
index d55868bc12d5da97a744ae3464c792305cb881d5..648f1667fc3d29db5fc53a51285b3849d035e86c 100644 (file)
@@ -304,13 +304,13 @@ namespace Ck { namespace IO {
       public:
         WriteSession(FileToken file_, size_t offset_, size_t bytes_)
           : file(CkpvAccess(manager)->get(file_))
-          , token(file_)
           , sessionOffset(offset_)
           , myOffset((sessionOffset / file->opts.peStripe + thisIndex)
                      * file->opts.peStripe)
           , sessionBytes(bytes_)
           , myBytes(min(file->opts.peStripe, sessionOffset + sessionBytes - myOffset))
           , myBytesWritten(0)
+          , token(file_)
         {
           CkAssert(file->fd != -1);
           CkAssert(myOffset >= sessionOffset);
index 209581865e4de099038c1612e21f17aceae5e454..659e8eaaf73738b43747931cadad2249baea652c 100644 (file)
@@ -86,7 +86,7 @@ gk_graph_t *gk_graph_Read(char *filename, int format, int isfewgts,
                 int isfvwgts, int isfvsizes)
 {
   ssize_t i, k, l;
-  size_t nfields, nvtxs, nedges, fmt, ncon, lnlen;
+  size_t nfields, nvtxs=0, nedges=0, fmt, ncon, lnlen;
   int32_t ival;
   float fval;
   int readsizes=0, readwgts=0, readvals=0, numbering=0;
index c81d61ebdbbd89acb2d16723a5a203e45a730ead..451d7b69b76510f3d349a175de507b05083644a1 100644 (file)
@@ -38,14 +38,14 @@ public:
 
 public:
   taskGraphSolver(CkArrayID set, int x)
-    : __taskDeps(), __taskSet(set), __taskIndex(CkArrayIndex1D(x)) {};
+    : __taskSet(set), __taskIndex(CkArrayIndex1D(x)), __taskDeps() {}
   taskGraphSolver(CkArrayID set, int x, int y)
-    : __taskDeps(), __taskSet(set), __taskIndex(CkArrayIndex2D(x,y)) {};
+    : __taskSet(set), __taskIndex(CkArrayIndex2D(x,y)), __taskDeps() {}
   taskGraphSolver(CkArrayID set, int x, int y, int z)
-    : __taskDeps(), __taskSet(set), __taskIndex(CkArrayIndex3D(x,y,z)) {};
+    : __taskSet(set), __taskIndex(CkArrayIndex3D(x,y,z)), __taskDeps() {}
   taskGraphSolver(CkArrayID set, CkArrayIndex taskIndex)
-    : __taskDeps(), __taskSet(set), __taskIndex(taskIndex) {};
-  taskGraphSolver(CkMigrateMessage *m) : PUP::able(m) {};
+    : __taskSet(set), __taskIndex(taskIndex), __taskDeps() {}
+  taskGraphSolver(CkMigrateMessage *m) : PUP::able(m) {}
 
   virtual void pup(PUP::er &p) {
   }
index bc110ee278142dd121b11c4cdcf2d1504217839a..c923498075761e9eff86232e9d7c56a0ceb629fd 100644 (file)
@@ -65,7 +65,7 @@ static void callWorker(void *msg)
     memcpy(m, r, size);
     CmiSetHandler(msg2, CpvAccess(responseHandler));
     CmiSyncSendAndFree(0, msgSize, msg2);
-};
+}
 
 static void response(void *msg)
 {
index ae1886c15ed11915387e712edfe28ee6f3ea87d4..1d6dc516292734e92f4404c2d38d9b3f39fee876 100644 (file)
@@ -32,8 +32,8 @@ class PUP_pagedDisk : public PUP::er {
        protected:
        pup_pagetable *_pagetable;
        void  *handle; // handle of the object to be restored
-       PUP_pagedDisk(unsigned int type,void *objhandle,pup_pagetable *pgtable):PUP::er(type),handle(objhandle),_pagetable(pgtable){
-       };
+       PUP_pagedDisk(unsigned int type,void *objhandle,pup_pagetable *pgtable):PUP::er(type),_pagetable(pgtable),handle(objhandle){
+       }
 
 };