dirsplit/dirflip: use constexpr rather than MACRO for defaults

axon10 · Nov 28, 2019 · a6611bd · a6611bd
1 parent af51eef
commit a6611bd
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 4 deletions.
diff --git a/cmd/dirflip.cpp b/cmd/dirflip.cpp
@@ -21,7 +21,8 @@
 #include "thread.h"
 #include "dwi/directions/file.h"
 
-#define DEFAULT_PERMUTATIONS 100000000
+constexpr size_t default_permutations = 1e8;
+
 
 
 using namespace MR;
@@ -153,7 +154,7 @@ void run ()
 {
   auto directions = DWI::Directions::load_cartesian (argument[0]);
 
-  size_t num_permutations = get_option_value<size_t> ("permutations", DEFAULT_PERMUTATIONS);
+  size_t num_permutations = get_option_value<size_t> ("permutations", default_permutations);
 
   vector<int> signs;
   {

diff --git a/cmd/dirsplit.cpp b/cmd/dirsplit.cpp
@@ -19,7 +19,7 @@
 #include "thread.h"
 #include "dwi/directions/file.h"
 
-#define DEFAULT_PERMUTATIONS 100000000
+constexpr size_t default_permutations = 1e8;
 
 
 using namespace MR;
@@ -170,7 +170,7 @@ void run ()
   if (num_subsets == 1)
     throw Exception ("Directions must be split across two or more output files");
 
-  const size_t num_permutations = get_option_value<size_t> ("permutations", DEFAULT_PERMUTATIONS);
+  const size_t num_permutations = get_option_value<size_t> ("permutations", default_permutations);
 
   vector<vector<size_t>> best;
   {