length from int to size_t
authorGengbin Zheng <gzheng@illinois.edu>
Thu, 5 Feb 2009 18:00:00 +0000 (18:00 +0000)
committerGengbin Zheng <gzheng@illinois.edu>
Thu, 5 Feb 2009 18:00:00 +0000 (18:00 +0000)
src/ck-com/ComlibArrayListener.h
src/ck-core/debug-charm.C
src/ck-core/debug-charm.h
src/ck-core/init.C
src/ck-core/register.C
src/ck-core/register.h
src/ck-perf/trace-common.C
src/conv-ccs/ccs-builtins.C
src/conv-core/debug-conv.h

index b5f28d07655e47d05cc228f7ffa66eaf597f724b..fc9a4a952837b6ace28bf7deda825759624657ba 100644 (file)
@@ -31,7 +31,7 @@ class ComlibArrayListener : public CkArrayListener{
     //remove strategy from table, and now it will not get updates
     //about this array
     void unregisterStrategy(StrategyTableEntry *entry) {
-        for(int count = 0; count < strategyList.size(); count++)
+        for(size_t count = 0; count < strategyList.size(); count++)
             if(strategyList[count] == entry)
                 strategyList.remove(count);
     }
index 644ddaed8faedc3b548a78293cf9be5ac352b503..9e45cbf6b60f15e31c92ce97cf0a0554a3fb98bd 100644 (file)
@@ -184,7 +184,7 @@ class CpdList_objectNames : public CpdListAccessor {
   int curGroup;
 public:
   virtual const char * getPath(void) const {return "charm/objectNames";}
-  virtual int getLength(void) const {
+  virtual size_t getLength(void) const {
     CkObjectRangeIterator<ignoreAdd> it(0,0,0);
     it.iterate();
     return it.getCount();
@@ -218,7 +218,7 @@ class CpdList_object : public CpdListAccessor {
   CpdListItemsRequest *reqq; // Only used while inside pup routine.
 public:
   virtual const char * getPath(void) const {return "charm/object";}
-  virtual int getLength(void) const {
+  virtual size_t getLength(void) const {
     CkObjectRangeIterator<ignoreAdd> it(0,0,0);
     it.iterate();
     return it.getCount();
@@ -254,7 +254,7 @@ class CpdList_arrayElementNames : public CpdListAccessor {
   PUP::er *pp; // Only used while inside pup routine.
 public:
   virtual const char * getPath(void) const {return "charm/arrayElementNames";}
-  virtual int getLength(void) const {
+  virtual size_t getLength(void) const {
     CkArrayElementRangeIterator<ignoreAdd> it(0,0,0);
     it.iterate();
     return it.getCount();
@@ -281,7 +281,7 @@ class CpdList_arrayElements : public CpdListAccessor {
   PUP::er *pp; // Only used while inside pup routine.
 public:
   virtual const char * getPath(void) const {return "charm/arrayElements";}
-  virtual int getLength(void) const {
+  virtual size_t getLength(void) const {
     CkArrayElementRangeIterator<ignoreAdd> it(0,0,0);
     it.iterate();
     return it.getCount();
@@ -380,7 +380,7 @@ class CpdList_localQ : public CpdListAccessor {
 public:
   CpdList_localQ() {}
   virtual const char * getPath(void) const {return "converse/localqueue";}
-  virtual int getLength(void) const {
+  virtual size_t getLength(void) const {
     int x = CdsFifo_Length((CdsFifo)(CpvAccess(debugQueue)));
     //CmiPrintf("*******Returning fifo length %d*********\n", x);
     //return CdsFifo_Length((CdsFifo)(CpvAccess(CmiLocalQueue)));
index 72bc4ce7cf4ec47a179ca3b0d092a964752e55ba..919fc535785ac4ce3af11dd072b1584d8e283bd3 100644 (file)
@@ -76,7 +76,7 @@ public:
   /// Return the CpdList path CCS clients should use to access this data.
   virtual const char *getPath(void) const =0;
   /// Return the length of this CpdList.
-  virtual int getLength(void) const =0;
+  virtual size_t getLength(void) const =0;
   /// Does this CpdList requires boundary checking?
   virtual bool checkBoundary(void) const { return checkBound; }
   /// Pup the items listed in this request.  Be sure to call beginItem between items!
@@ -106,7 +106,7 @@ public:
   void operator=(const CpdListAccessor_c &p);  // You don't want to copy
   
   virtual const char *getPath(void) const {return path;}
-  virtual int getLength(void) const {return (*lenFn)(lenParam);}
+  virtual size_t getLength(void) const {return (*lenFn)(lenParam);}
   virtual void pup(PUP::er &p,CpdListItemsRequest &req) {
     (itemsFn)(itemsParam,(pup_er *)&p,&req);
   }
@@ -124,7 +124,7 @@ public:
        typedef void (*pupFn)(PUP::er &p,int itemNo);
 private:
        const char *path;
-       int &length;
+       size_t &length;
        pupFn pfn;
 public:
        /**
@@ -136,11 +136,11 @@ public:
                            In particular, this means length must not be moved!
             \param pfn_ Function to pup the items in the list.
        */
-       CpdSimpleListAccessor(const char *path_,int &length_,pupFn pfn_)
+       CpdSimpleListAccessor(const char *path_,size_t &length_,pupFn pfn_)
                :path(path_),length(length_),pfn(pfn_) { }
        virtual ~CpdSimpleListAccessor();
        virtual const char *getPath(void) const;
-       virtual int getLength(void) const;
+       virtual size_t getLength(void) const;
        virtual void pup(PUP::er &p,CpdListItemsRequest &req);
 };
 
index d00ed32429ee0c2a50d21af72fa76d3843f02e70..9c05d09d842edb6c62b3ff05b7cf3be6d7bc2ec4 100644 (file)
@@ -475,7 +475,7 @@ static inline void _processRODataMsg(envelope *env)
 {
   //Unpack each readonly:
   PUP::fromMem pu((char *)EnvToUsr(env));
-  for(int i=0;i<_readonlyTable.size();i++) _readonlyTable[i]->pupData(pu);
+  for(size_t i=0;i<_readonlyTable.size();i++) _readonlyTable[i]->pupData(pu);
   CmiFree(env);
 }
 
@@ -883,7 +883,7 @@ void _initCharm(int unused_argc, char **argv)
                 }
        }else if(CkMyPe()==0){
                _allStats = new Stats*[CkNumPes()];
-               register int i, nMains=_mainTable.size();
+               register size_t i, nMains=_mainTable.size();
                for(i=0;i<nMains;i++)  /* Create all mainchares */
                {
                        register int size = _chareTable[_mainTable[i]->chareIdx]->size;
index 5ba7f86d5dfa1dcf56ae8305a70fdbc458d0e4c0..d826cf251ebbc6eab3470e7dfea1d0d2853ea029 100644 (file)
@@ -206,7 +206,7 @@ void _registerDone(void)
 
 //Print a debugging version of this entry method index:
 void CkPrintEntryMethod(int epIdx) {
-       if (epIdx<=0 || epIdx>=_entryTable.size())
+       if (epIdx<=0 || epIdx>=(int)_entryTable.size())
                CkPrintf("INVALID ENTRY METHOD %d!",epIdx);
        else {
                EntryInfo *e=_entryTable[epIdx];
@@ -217,7 +217,7 @@ void CkPrintEntryMethod(int epIdx) {
 
 //Print a debugging version of this chare index:
 void CkPrintChareName(int chareIdx) {
-       if (chareIdx<=0 || chareIdx>=_chareTable.size())
+       if (chareIdx<=0 || chareIdx>=(int)_chareTable.size())
                CkPrintf("INVALID CHARE INDEX %d!",chareIdx);
        else {
                ChareInfo *c=_chareTable[chareIdx];
index ee2282960f1385e542799a62d3105b482e9951cf..638a3ffd787e8bb897ee145e6bf7061b7c9b555e 100644 (file)
@@ -250,7 +250,7 @@ public:
        */
        CkRegisteredInfo() :vec(CkSkipInitialization()) {}
         ~CkRegisteredInfo() {
-               for (int i=0; i<vec.size(); i++) if (vec[i]) delete vec[i];
+               for (size_t i=0; i<vec.size(); i++) if (vec[i]) delete vec[i];
        } 
 
        /// Add a heap-allocated registration record,
@@ -267,10 +267,10 @@ public:
        
        /// Return the number of registered entities in this table.
        /// (This is a reference so the CpdLists can stay up to date).
-       int &size(void) {return vec.length();}
+       size_t &size(void) {return vec.length();}
        
        /// Return the registered data at this index.
-       T *operator[](int idx) {
+       T *operator[](size_t idx) {
 #ifndef CMK_OPTIMIZE
                /* Bounds-check the index: */
                if (idx<0 || idx>=vec.size()) outOfBounds(idx);
index b5df5d02abb4293def88806f2b0aa18bda0dd7dc..0f243eaa9f07bee271b25fbfa0b32da5bebefa46 100644 (file)
@@ -132,7 +132,6 @@ static void traceCommonInit(char **argv)
        char *cwd;
   CkpvInitialize(char*, selective);
   if (CmiGetArgStringDesc(argv, "+selective", &temproot, "TAU's selective instrumentation file")) {
-    int i;
     // Trying to decide if the traceroot path is absolute or not. If it is not
     // then create an absolute pathname for it.
     if (temproot[0] != PATHSEP) {
@@ -193,7 +192,7 @@ extern void traceWriteSTS(FILE *stsfp,int nUserEvents) {
   fprintf(stsfp, "TOTAL_MSGS %d\n", _msgTable.size());
   fprintf(stsfp, "TOTAL_PSEUDOS %d\n", 0);
   fprintf(stsfp, "TOTAL_EVENTS %d\n", nUserEvents);
-  int i;
+  size_t i;
   for(i=0;i<_chareTable.size();i++)
     fprintf(stsfp, "CHARE %d %s\n", i, _chareTable[i]->name);
   for(i=0;i<_entryTable.size();i++)
@@ -258,7 +257,6 @@ extern "C" void traceEnd(void) {
 
 static int checkTraceOnPe(char **argv)
 {
-  int i;
   int traceOnPE = 1;
   char *procs = NULL;
 #if CMK_BLUEGENE_CHARM
@@ -400,7 +398,7 @@ extern "C"
 void traceMemoryUsage()
 {
 #ifndef CMK_OPTIMIZE
-  long d = CmiMemoryUsage();
+  double d = CmiMemoryUsage()*1.0;
 
   if (CpvAccess(traceOn) && CkpvAccess(_traces))
     CkpvAccess(_traces)->memoryUsage(d);
index bc1a0041626f465b8b368d2a013532f93059a7c5..726fe723635e45775b0c44354c0018f837047c95 100644 (file)
@@ -298,8 +298,8 @@ class CpdList_introspect : public CpdListAccessor {
 public:
   CpdList_introspect(CpdListTable_t *tab_) :tab(tab_) { }
   virtual const char *getPath(void) const { return "converse/lists";}
-  virtual int getLength(void) const {
-    int len=0;
+  virtual size_t getLength(void) const {
+    size_t len=0;
     CkHashtableIterator *it=tab->iterator();
     while (NULL!=it->next()) len++;
     delete it;
@@ -332,7 +332,7 @@ disabled, to avoid bizarre link-time errors.*/
 CpdListAccessor::~CpdListAccessor() { }
 CpdSimpleListAccessor::~CpdSimpleListAccessor() { }
 const char *CpdSimpleListAccessor::getPath(void) const {return path;}
-int CpdSimpleListAccessor::getLength(void) const {return length;}
+size_t CpdSimpleListAccessor::getLength(void) const {return length;}
 void CpdSimpleListAccessor::pup(PUP::er &p,CpdListItemsRequest &req) 
 {
        for (int i=req.lo;i<req.hi;i++) {
index 9d56a243935f5cfe855381c0e68205e7eff97498..5f743eb9b3105522b25890b6661c6d6c27d9d6c8 100644 (file)
@@ -79,7 +79,7 @@ typedef void (*CpdListItemsFn_c)(void *itemsParam,pup_er p,
     \param lenParam User-defined parameter passed to CpdListRegister_c.
     \param return Length of the CpdList.
 */
-typedef int  (*CpdListLengthFn_c)(void *lenParam);
+typedef size_t  (*CpdListLengthFn_c)(void *lenParam);
 
 /**
   Create a new CpdList at the given path.  When a CCS client requests