renamed numElements in ArrayElement to _numElements.
authorGengbin Zheng <gzheng@newskillz.(none)>
Wed, 3 Mar 2010 22:00:46 +0000 (16:00 -0600)
committerGengbin Zheng <gzheng@newskillz.(none)>
Wed, 3 Mar 2010 22:00:46 +0000 (16:00 -0600)
numElements is a populate variable name, which is error prone to be a class member. e.g. when there is a global variable with same name.

src/ck-core/ckarray.C
src/ck-core/ckarray.h
src/ck-core/tempo.C
src/libs/ck-libs/ampi/ampi.C
src/libs/ck-libs/irecv/receiver.h
src/libs/ck-libs/mblock/mblock.C

index 2abc9b6a09bd3c12dfdccbf12a08255f7068387a..fe3160e0aede64dd6cd46fa1bc14e875b2cf449d 100644 (file)
@@ -184,7 +184,7 @@ void ArrayElement::initBasics(void)
   ArrayElement_initInfo &info=CkpvAccess(initInfo);
   thisArray=info.thisArray;
   thisArrayID=info.thisArrayID;
-  numElements=info.numInitial.getCombinedCount();
+  _numElements=info.numInitial.getCombinedCount();
   if (info.listenerData) {
     memcpy(listenerData,info.listenerData,sizeof(listenerData));
   }
index ea4e8eb08f2f129672d8a24e9ef837d7cfd6e4da..fd9b6cfca3c035bbb4345aa4ea431dafc354945a 100644 (file)
@@ -539,7 +539,7 @@ public:
   ArrayElement(CkMigrateMessage *m);
   virtual ~ArrayElement();
 
-  int numElements; /// Initial number of array elements (DEPRICATED)
+  int _numElements; /// Initial number of array elements (DEPRICATED)
   // On the previous line, someone wrote "deprecated", but nevertheless it is still
   // used on TempoArray (tempo.C), ampi.C, irecv (receiver.h), as well as many tests and examples!
 
index 01069ac7b26a1074544c1df4105b6525c2fea4b0..137da62452d2dbb8d9d8745e3fb12fda95a2621e 100644 (file)
@@ -163,9 +163,9 @@ TempoArray::ckTempoBarrier(void)
     ckTempoRecv(BARR_TAG, nGOps, (void*) 0, 0);
   } else {
      int i;
-     for(i=1;i<numElements;i++)
+     for(i=1;i<_numElements;i++)
        ckTempoRecv(BARR_TAG, nGOps, (void *) 0, 0);
-     for(i=1;i<numElements;i++)
+     for(i=1;i<_numElements;i++)
        ckTempoSendElem(BARR_TAG, nGOps, (void *) 0, 0, i);
   }
   nGOps++;
@@ -176,7 +176,7 @@ TempoArray::ckTempoBcast(int sender, int tag, void *buffer, int buflen)
 {
   if(sender) {
     int i;
-    for(i=0;i<numElements;i++)
+    for(i=0;i<_numElements;i++)
       ckTempoSendElem(tag, BCAST_TAG+nGOps, buffer, buflen, i);
   }
   ckTempoRecv(tag, BCAST_TAG+nGOps, buffer, buflen);
@@ -248,7 +248,7 @@ TempoArray::ckTempoReduce(int root, int op, void *inbuf, void *outbuf,
     memcpy(outbuf, inbuf, size);
     void *tbuf = malloc(size);
     _MEMCHECK(tbuf);
-    for(int i=0; i<numElements-1; i++) {
+    for(int i=0; i<_numElements-1; i++) {
       ckTempoRecv(REDUCE_TAG, nGOps, tbuf, size);
       doOp(op, type, count, tbuf, outbuf);
     }
index 279dfa1c88e9ddde6546dee361ac8a8c1af861be..620265d32e24640ba650a12079aecf6b873013f3 100644 (file)
@@ -1122,7 +1122,7 @@ ampi::ampi(CkArrayID parent_,const ampiCommStruct &s)
 
   comlibProxy = thisProxy; // Will later be associated with comlib
   
-  seqEntries=parent->numElements;
+  seqEntries=parent->_numElements;
   oorder.init (seqEntries);
 #ifdef _FAULT_MLOG_
     if(thisIndex == 0){
@@ -1158,7 +1158,7 @@ ampi::ampi(CkArrayID parent_,const ampiCommStruct &s, ComlibInstanceHandle ciStr
   //  ComlibAssociateProxy(ciAlltoall, comlibProxy);
 #endif
 
-  seqEntries=parent->numElements;
+  seqEntries=parent->_numElements;
   oorder.init (seqEntries);
 }
 
index f987035ca877ee6be6df46152aa7897b364ddd42..03841ed9ea2dfc19b016fd7df861bb43dcd6c1fc 100644 (file)
@@ -76,7 +76,7 @@ public:
   receiver(CkMigrateMessage *);
   ~receiver();
 
-  int getArraySize(void) const {return numElements;}
+  int getArraySize(void) const {return _numElements;}
 
   // interface
   void isend(void *buf, int count, int datatype, int dest, int tag, int refno);
index 3b8f377a06fb87c03532e3d74ae57b0b8dcd0866..4ceaaa44f3f3ce97996643dc11e5fe70858223f3 100644 (file)
@@ -424,7 +424,7 @@ MBlockChunk::reduce(int fid, const void *inbuf, void *outbuf, int op)
 {
   const DType &dt = fields[fid]->dt;
   int len = dt.length();
-  if(numElements==1) {
+  if(_numElements==1) {
     memcpy(outbuf,inbuf,len);
     return;
   }