use .data() instead of &v[0] to avoid annoying analyzers.

diff --git a/src/smpi/colls/smpi_default_selector.cpp b/src/smpi/colls/smpi_default_selector.cpp
index d5f5aac..8d449de 100644 (file)
--- a/src/smpi/colls/smpi_default_selector.cpp
+++ b/src/smpi/colls/smpi_default_selector.cpp
@@ -78,7 +78,7 @@ int allgatherv__default(const void *sendbuf, int sendcount, MPI_Datatype sendtyp
   MPI_Request request;
   colls::iallgatherv(sendbuf, sendcount, sendtype, recvbuf, recvcounts, displs, recvtype, comm, &request, 0);
   auto requests = request->get_nbc_requests();
-  Request::waitall(requests.size(), &requests[0], MPI_STATUS_IGNORE);
+  Request::waitall(requests.size(), requests.data(), MPI_STATUS_IGNORE);
   for(auto& req: requests)
     Request::unref(&req);
   Request::unref(&request);

diff --git a/src/smpi/mpi/smpi_request.cpp b/src/smpi/mpi/smpi_request.cpp
index f6783a9..0b0c740 100644 (file)
--- a/src/smpi/mpi/smpi_request.cpp
+++ b/src/smpi/mpi/smpi_request.cpp
@@ -854,9 +854,9 @@ int Request::finish_nbc_requests(MPI_Request* request, int test){
   int flag = 1;
   int ret = 0;
   if(test == 0)
-    ret = waitall((*request)->nbc_requests_.size(), &(*request)->nbc_requests_[0], MPI_STATUSES_IGNORE);
+    ret = waitall((*request)->nbc_requests_.size(), (*request)->nbc_requests_.data(), MPI_STATUSES_IGNORE);
   else{
-    ret = testall((*request)->nbc_requests_.size(), &(*request)->nbc_requests_[0], &flag, MPI_STATUSES_IGNORE);
+    ret = testall((*request)->nbc_requests_.size(), (*request)->nbc_requests_.data(), &flag, MPI_STATUSES_IGNORE);
   }
   if(ret!=MPI_SUCCESS)
     xbt_die("Failure when waiting on non blocking collective sub-requests");
@@ -1263,7 +1263,7 @@ int Request::grequest_complete(MPI_Request request)
 void Request::start_nbc_requests(std::vector<MPI_Request> reqs){
   if (reqs.size() > 0) {
     nbc_requests_ = reqs;
-    Request::startall(reqs.size(), &reqs[0]);
+    Request::startall(reqs.size(), reqs.data());
   }
 }