Refactor main and fix gcc warning

kennethassogba · Nov 16, 2024 · 2ed6abc · 2ed6abc
1 parent f0ab66a
commit 2ed6abc
Show file tree

Hide file tree

Showing 2 changed files with 10 additions and 8 deletions.
diff --git a/examples/main.cpp b/examples/main.cpp
@@ -4,16 +4,18 @@
 #include <Eigen/Dense>
 #include "demeter/model/material.hpp"
 
-int main() {
+void CheckOpenMP() {
 #if defined(_OPENMP)
-  std::cerr << "OpenMP version " << _OPENMP << '\n';
+  unsigned num_threads = 0;
+#pragma omp parallel reduction(+ : num_threads)
+  num_threads += 1;
+  std::cerr << "OpenMP version " << _OPENMP << " number of threads "
+            << num_threads << '\n';
 #endif
+}
 
-#pragma omp parallel for
-  for (int i = 0; i < 9; ++i) {
-    std::cerr << "rank = " << omp_get_thread_num()
-              << " size = " << omp_get_num_threads() << '\n';
-  }
+int main() {
+  CheckOpenMP();
 
   using Eigen::MatrixXd;
 

diff --git a/src/demeter/model/material.hpp b/src/demeter/model/material.hpp
@@ -55,7 +55,7 @@ class Material {
  private:
   void check() const;
   size_t check(size_t group) const {
-    assert((group >= 0) and (group < num_groups_));
+    assert(group < num_groups_);
     return group;
   }