Logo AND Algorithmique Numérique Distribuée

Public GIT Repository
[SMPI/MC] Use std::strlen() instead of manual counting.
authorChristian Heinrich <franz-christian.heinrich@inria.fr>
Thu, 16 Jun 2016 13:22:43 +0000 (15:22 +0200)
committerChristian Heinrich <franz-christian.heinrich@inria.fr>
Fri, 17 Jun 2016 09:27:40 +0000 (11:27 +0200)
src/mc/mc_checkpoint.cpp
src/smpi/smpi_bench.cpp

index 20754d3..d6e234c 100644 (file)
@@ -527,7 +527,7 @@ static std::vector<s_fd_infos_t> get_current_fds(pid_t pid)
 
     // We don't handle them.
     // It does not mean we should silently ignore them however.
-    if (strncmp(link, "pipe:", 5) == 0 || strncmp(link, "socket:", 7) == 0)
+    if (strncmp(link, "pipe:", std::strlen("pipe:")) == 0 || strncmp(link, "socket:", std::strlen("socket:")) == 0)
       continue;
 
     // If dot_output enabled, do not handle the corresponding file
@@ -541,7 +541,7 @@ static std::vector<s_fd_infos_t> get_current_fds(pid_t pid)
     }
 
     // This is probably a shared memory used by lttng-ust:
-    if(strncmp("/dev/shm/ust-shm-tmp-", link, 21)==0)
+    if(strncmp("/dev/shm/ust-shm-tmp-", link, std::strlen("/dev/shm/ust-shm-tmp-"))==0)
       continue;
 
     // Add an entry for this FD in the snapshot:
index e6998c3..2aa1b81 100644 (file)
@@ -646,7 +646,7 @@ void smpi_really_switch_data_segment(int dest) {
 
 int smpi_is_privatisation_file(char* file)
 {
-  return strncmp("/dev/shm/my-buffer-", file, 19) == 0;
+  return strncmp("/dev/shm/my-buffer-", file, std::strlen("/dev/shm/my-buffer-")) == 0;
 }
 
 void smpi_initialize_global_memory_segments(){