Fix error with if(string)

CLIUtils · Nov 28, 2017 · 7726dc2 · 7726dc2
1 parent 8be496e
commit 7726dc2
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 2 deletions.
diff --git a/include/CLI/App.hpp b/include/CLI/App.hpp
@@ -589,7 +589,7 @@ class App {
             remove_option(config_ptr_);
 
         // Only add config if option passed
-        if(name) {
+        if(!name.empty()) {
             config_name_ = default_filename;
             config_required_ = required;
             config_ptr_ = add_option(name, config_name_, help, !default_filename.empty());

diff --git a/tests/IniTest.cpp b/tests/IniTest.cpp
@@ -429,7 +429,8 @@ TEST_F(TApp, IniSubFailure) {
 
     EXPECT_THROW(run(), CLI::INIError);
 }
-d TEST_F(TApp, IniNoSubFailure) {
+
+TEST_F(TApp, IniNoSubFailure) {
 
     TempFile tmpini{"TestIniTmp.ini"};