Merge pull request #413 from SCOREC/apw/fix-capvol-warnings

Fix test/capVol CMake and compiler warnings
SCOREC · Jan 5, 2024 · 604c067 · 604c067
2 parents 50567e5 + a638c96
commit 604c067
Show file tree

Hide file tree

Showing 2 changed files with 1 addition and 5 deletions.
diff --git a/test/CMakeLists.txt b/test/CMakeLists.txt
@@ -211,7 +211,7 @@ endif()
 
 if(ENABLE_CAPSTONE)
   util_exe_func(capVol capVol.cc)
-  target_include_directories(capVol PUBLIC "${PROJECT_SOURCE_DIR}/capstone_clis")
+  target_include_directories(capVol PRIVATE "${PROJECT_SOURCE_DIR}/capstone_clis")
 endif()
 
 # send all the newly added utility executable targets

diff --git a/test/capVol.cc b/test/capVol.cc
@@ -95,13 +95,9 @@ int main(int argc, char** argv) {
   bool volume_flag = false, write_flag = false, analytic_flag = false,
        verbose_flag = false;
   for (int i = 1; i < argc - 2; ++i) {
-    bool flagDone = false;
     if (*argv[i] == '-') {
       for (int j = 1; argv[i][j] != '\0'; ++j) {
         switch(argv[i][j]) {
-        case '-':
-          flagDone = true;
-          break;
         case 'a':
           analytic_flag = true;
           break;