For loop replaced with for_each in several places in parallelization.

src/Common/src/MatrixSeries.cpp

@@ -146,10 +146,10 @@ namespace FenestrationCommon
         {
             FenestrationCommon::executeInParallel<size_t>(0, matrix.size() - 1, [&](size_t i) {
                 const auto & row = matrix[i];
-                for(size_t j = 0; j < row.size(); ++j)
-                {
+                std::for_each(begin(row), end(row), [&](const auto & elem) {
+                    size_t j = &elem - &row[0];
                     func(i, j);
-                }
+                });
             });
         }
     }   // namespace

src/Common/src/Parallel.hpp

@@ -6,32 +6,32 @@
 
 #include "Utility.hpp"
 
-namespace FenestrationCommon {
+namespace FenestrationCommon
+{
     template<typename IndexType, typename Function>
-    void executeInParallel(IndexType start, IndexType end, Function&& func)
+    void executeInParallel(IndexType start, IndexType end, Function && func)
     {
         const auto numberOfThreads = getNumberOfThreads(end - start + 1);
         const auto chunks = chunkIt(start, end, numberOfThreads);
 
         std::vector<std::thread> workers;
         workers.reserve(chunks.size());
 
-        for (const auto& chunk : chunks)
+        for(const auto & chunk : chunks)
         {
             workers.emplace_back([&, chunk]() {
-                for (IndexType i = chunk.start; i < chunk.end; ++i)
+                for(IndexType i = chunk.start; i < chunk.end; ++i)
                 {
                     func(i);
                 }
             });
         }
 
-        for (auto& worker : workers)
-        {
-            if (worker.joinable())
+        std::for_each(workers.begin(), workers.end(), [](std::thread & worker) {
+            if(worker.joinable())
             {
                 worker.join();
             }
-        }
+        });
     }
-}
+}   // namespace FenestrationCommon