ckarrayindexmax: Delete class and make the name synonymous with base class
authorRamprasad Venkataraman <ramv@illinois.edu>
Tue, 31 May 2011 21:25:19 +0000 (16:25 -0500)
committerRamprasad Venkataraman <ramv@illinois.edu>
Wed, 8 Jun 2011 15:37:39 +0000 (10:37 -0500)
CkArrayIndexMax existed because charm permitted array index objects of
different sizes and the Max object could hold the largest of them.  However,
recently charm has moved to support only a single-sized array index object and
hence the Max class has become vestigial.

Delete the class, and typedef CkArrayIndexMax to be synonymous with CkArrayIndex.
Code should eventually be cleaned to replace all references to Max, but the
typedef exists until then.

src/ck-core/charm++.h
src/ck-core/charm.h
src/ck-core/ckarray.C
src/libs/ck-libs/multicast/ckmulticast.h

index 851e6be73cb5039fb624559645dc2358b677e0a7..ee494c3d2a020f75485a1e281968f0b2d4385ae8 100644 (file)
@@ -298,6 +298,18 @@ public:
     void print() { CmiPrintf("%d: %d %d %d\n", nInts,index[0], index[1], index[2]); }
 };
 
+/**
+ * Support the large body of code that still uses the
+ * (now dead) CkArrayIndexMax class to manipulate array
+ * indices. All the functionality is now incorporated
+ * into the CkArrayIndex base class itself.
+ *
+ * It is recommended that newer code directly use the
+ * base class when there is need to handle an array
+ * index.
+ */
+typedef CkArrayIndex CkArrayIndexMax;
+
 inline CkHashCode CkArrayIndex::hash(void) const
 {
         register int i;
@@ -326,18 +338,6 @@ inline int CkArrayIndex::compare(const CkArrayIndex &i2) const
 }
 
 
-//This class is as large as any CkArrayIndex
-class CkArrayIndexMax : public CkArrayIndex {
-public:
-       CkArrayIndexMax(void)  { init(); }
-       CkArrayIndexMax(int i) { init(); }
-       CkArrayIndexMax(const CkArrayIndex &that) 
-               {copyFrom(that);}
-       CkArrayIndexMax &operator=(const CkArrayIndex &that) 
-               {copyFrom(that); return *this;}
-};
-PUPmarshall(CkArrayIndexMax)
-
 
 class CkArrayID {
        CkGroupID _gid;
index b4284ebc736308936b8a737b54063a4577877578..3d026d709323318579366149d5c4f2f711d3d2c1 100644 (file)
@@ -348,10 +348,10 @@ extern void  CkSendToFutureID(CkFutureID futNum, void *msg, int pe);
 extern CkFutureID CkCreateAttachedFuture(void *msg);
 /* forward declare */
 struct CkArrayID;
-struct CkArrayIndexMax;
+struct CkArrayIndex;
 extern CkFutureID CkCreateAttachedFutureSend(void *msg, int ep,
-struct CkArrayID id, struct CkArrayIndexMax idx, void(*fptr)(struct
-CkArrayID,struct CkArrayIndexMax,void*,int,int),int size CK_MSGOPTIONAL);
+struct CkArrayID id, CkArrayIndex idx, void(*fptr)(struct
+CkArrayID, CkArrayIndex,void*,int,int),int size CK_MSGOPTIONAL);
 /* extern CkFutureID CkCreateAttachedFutureSend(void *msg, int ep, void*,void(*fptr)(void*,void*,int,int)); */
 
 extern void *CkWaitReleaseFuture(CkFutureID futNum);
index d08960b67b1686fa21441337f2fa3c0062bfe5bc..d61949f46c88dd709e2d695ffe8639575dd8aed7 100644 (file)
@@ -426,7 +426,7 @@ CkLocMgr *CProxy_ArrayBase::ckLocMgr(void) const
 CK_REDUCTION_CLIENT_DEF(CProxy_ArrayBase,ckLocalBranch())
 
 CkArrayOptions::CkArrayOptions(void) //Default: empty array
-       :numInitial(0),map(_defaultArrayMapID)
+       :numInitial(),map(_defaultArrayMapID)
 {
     init();
 }
index 506b2c9dd13fb3ecbc4c8243439ca8a06d647868..b6567dbe7ca3bac794f68ea993d1f0b91e0a82c0 100644 (file)
@@ -39,7 +39,7 @@ class CkMulticastMgr: public CkDelegateMgr
                 int  pe;
             public:
                 IndexPos() {}
-                IndexPos(int i): idx(i), pe(i) {}
+                IndexPos(int i): idx(), pe(i) {}
                 IndexPos(CkArrayIndexMax i, int p): idx(i), pe(p) {};
                 void pup(PUP::er &p){ p|idx; p|pe; }
         };