Don't do 'using namespace std;' in an MSA header
authorPhil Miller <mille121@illinois.edu>
Tue, 3 Feb 2009 22:29:05 +0000 (16:29 -0600)
committerPhil Miller <mille121@illinois.edu>
Thu, 10 Dec 2009 22:22:33 +0000 (16:22 -0600)
Add std:: qualification to the 4 lines where it's necessary, rather
than dumping all of the standard namespace into scope in a header.

src/libs/ck-libs/multiphaseSharedArrays/msa-DistPageMgr.h

index e65e942a26fe1b22a877b3720943d659b0493c80..36cfe3255592b248cc85c40a282c86d31f667c83 100644 (file)
@@ -26,8 +26,6 @@ template <class ENTRY, class MERGER,
          class MSA_PageT;
 #include "msa.decl.h"
 
          class MSA_PageT;
 #include "msa.decl.h"
 
-using namespace std;
-
 inline int mymin(int a, int b)
 {
     return (a<b)?a:b;
 inline int mymin(int a, int b)
 {
     return (a<b)?a:b;
@@ -252,7 +250,7 @@ protected:
     unsigned int nPages;            // number of pages
     const page_ptr_t* pageTable; // actual data for pages (NULL means page is gone)
     MSA_Page_State** pageState;  // state of each page
     unsigned int nPages;            // number of pages
     const page_ptr_t* pageTable; // actual data for pages (NULL means page is gone)
     MSA_Page_State** pageState;  // state of each page
-    list<unsigned int> stackOfPages;
+    std::list<unsigned int> stackOfPages;
     unsigned int lastPageAccessed;
 
 public:
     unsigned int lastPageAccessed;
 
 public:
@@ -266,7 +264,7 @@ public:
             lastPageAccessed = page;
 
             // delete this page from the stack and push it at the top
             lastPageAccessed = page;
 
             // delete this page from the stack and push it at the top
-            list<unsigned int>::iterator i;
+                       std::list<unsigned int>::iterator i;
             for(i = stackOfPages.begin(); i != stackOfPages.end(); i++)
                 if(*i == page)
                     i = stackOfPages.erase(i);
             for(i = stackOfPages.begin(); i != stackOfPages.end(); i++)
                 if(*i == page)
                     i = stackOfPages.erase(i);
@@ -281,7 +279,7 @@ public:
             return MSA_INVALID_PAGE_NO;
 
         // find a non-empty unlocked page to swap, delete all empty pages from the stack
             return MSA_INVALID_PAGE_NO;
 
         // find a non-empty unlocked page to swap, delete all empty pages from the stack
-        list<unsigned int>::iterator i = stackOfPages.begin();
+               std::list<unsigned int>::iterator i = stackOfPages.begin();
         while(i != stackOfPages.end())
         {
             if(pageTable[*i] == NULL) i = stackOfPages.erase(i);
         while(i != stackOfPages.end())
         {
             if(pageTable[*i] == NULL) i = stackOfPages.erase(i);
@@ -522,7 +520,7 @@ protected:
                        l->suspend();
     }
 
                        l->suspend();
     }
 
-    stack<ENTRY_TYPE*> pagePool;     // a pool of unused pages
+    std::stack<ENTRY_TYPE*> pagePool;     // a pool of unused pages
     
     typedef vmNRUReplacementPolicy vmPageReplacementPolicy;
     vmPageReplacementPolicy* replacementPolicy;
     
     typedef vmNRUReplacementPolicy vmPageReplacementPolicy;
     vmPageReplacementPolicy* replacementPolicy;
@@ -531,7 +529,7 @@ protected:
     typedef struct { unsigned int begin; unsigned int end; } writebounds_t;
 
     // a list of write bounds associated with a given page
     typedef struct { unsigned int begin; unsigned int end; } writebounds_t;
 
     // a list of write bounds associated with a given page
-    typedef list<writebounds_t> writelist_t;
+    typedef std::list<writebounds_t> writelist_t;
 
     writelist_t** writes;           // the write lists for each page
 
 
     writelist_t** writes;           // the write lists for each page