Use .data() instead of &vector[0]

Author: aledomu

classdescMP.h

@@ -186,15 +186,15 @@ namespace classdesc
     {
       int flag;
       for (size_t i=0; i<bufs.size(); i++) requests[i]=bufs[i].request;
-      MPI_Testall(bufs.size(),&requests[0],&flag,MPI_STATUSES_IGNORE);
+      MPI_Testall(bufs.size(),requests.data(),&flag,MPI_STATUSES_IGNORE);
       return flag;
     }
     /// return the index of any request that has completed, or MPI_UNDEFINED if none  
     int testany()
     {
       int flag,index;
       for (size_t i=0; i<bufs.size(); i++) requests[i]=bufs[i].request;
-      MPI_Testany(bufs.size(),&requests[0],&index,&flag,MPI_STATUS_IGNORE);
+      MPI_Testany(bufs.size(),requests.data(),&index,&flag,MPI_STATUS_IGNORE);
       return index;
     }
     /// return the index of the requests that have completed
@@ -203,21 +203,21 @@ namespace classdesc
       int count;
       std::vector<int> index(bufs.size());
       for (size_t i=0; i<bufs.size(); i++) requests[i]=bufs[i].request;
-      MPI_Testsome(bufs.size(),&requests[0],&count,&index[0],MPI_STATUSES_IGNORE);
+      MPI_Testsome(bufs.size(),requests.data(),&count,index.data(),MPI_STATUSES_IGNORE);
       return std::vector<int>(index.begin(),index.begin()+count);
     }
     /// wait for all outstanding requests to complete
     void waitall() 
     {
       for (size_t i=0; i<bufs.size(); i++) requests[i]=bufs[i].request;
-      MPI_Waitall(bufs.size(),&requests[0],MPI_STATUSES_IGNORE);
+      MPI_Waitall(bufs.size(),requests.data(),MPI_STATUSES_IGNORE);
     }
     /// wait for any outstanding request to complete, returning index of completed request
     int waitany()
     {
       int index;
       for (size_t i=0; i<bufs.size(); i++) requests[i]=bufs[i].request;
-      MPI_Waitany(bufs.size(),&requests[0],&index,MPI_STATUSES_IGNORE);
+      MPI_Waitany(bufs.size(),requests.data(),&index,MPI_STATUSES_IGNORE);
       return index;
     }
     /// wait for some outstanding requests to complete, returning an array of request indices
@@ -226,7 +226,7 @@ namespace classdesc
       int count;
       std::vector<int> index(bufs.size());
       for (size_t i=0; i<bufs.size(); i++) requests[i]=bufs[i].request;
-      MPI_Waitsome(bufs.size(),&requests[0],&count,&index[0],MPI_STATUS_IGNORE);
+      MPI_Waitsome(bufs.size(),requests.data(),&count,index.data(),MPI_STATUS_IGNORE);
       return std::vector<int>(index.begin(),index.begin()+count);
     }
   };

java/JNIEx.h

@@ -81,7 +81,7 @@ namespace classdesc
         objName << std::hex << cleanupContainer.back().get();
         std::string objNameStr=objName.str();
         std::vector<jchar> jname(objNameStr.begin(), objNameStr.end());
-        env->SetObjectField(obj,fld,env->NewString(&jname[0], jname.size()));
+        env->SetObjectField(obj,fld,env->NewString(jname.data(), jname.size()));
         // register the objects instance variables and methods with the
         // method registry
         register_java_class jc;

javaClass_base.h

@@ -209,7 +209,7 @@ namespace classdesc
     static jstring from(JNIEnv *env, const char* x) 
     {
       std::vector<jchar> tmp(x, x+strlen(x));
-      return env->NewString(&tmp[0], strlen(x));
+      return env->NewString(tmp.data(), strlen(x));
     }
     static jstring from(JNIEnv *env, const std::string& x) 
     {

pack_stl.h

@@ -82,7 +82,7 @@ namespace classdesc
   {
     (*this) << uint64(x.size());
     if (!x.empty())
-      packer.packraw(reinterpret_cast<const char*>(&x[0]), 
+      packer.packraw(reinterpret_cast<const char*>(x.data()), 
                      x.size()*sizeof(x[0]));
     return *this;
   }
@@ -95,7 +95,7 @@ namespace classdesc
       throw pack_error("invalid size for data available");
     x.resize(sz);
     if (sz)
-      packer.unpackraw(reinterpret_cast<char*>(&x[0]), 
+      packer.unpackraw(reinterpret_cast<char*>(x.data()), 
                        x.size()*sizeof(x[0]));
     return *this;
   }
@@ -253,15 +253,15 @@ namespace classdesc_access
     void asg(string& x, const std::vector<cT>& b) 
     {
       if (!b.empty())
-        x=string(&b[0], b.size()-1);
+        x=string(b.data(), b.size()-1);
     }
 
     template <class U>
     void operator()(classdesc::pack_t& targ, const classdesc::string& desc, U& arg) 
     {
       classdesc::uint64 size=0; targ>>size;
       std::vector<cT> buf(size+1); //ensure buf[0] exists
-      targ.unpackraw(&buf[0],sizeof(cT)*size);
+      targ.unpackraw(buf.data(),sizeof(cT)*size);
       asg(arg, buf);
     }
   };