diff --git a/include/CLI/App.hpp b/include/CLI/App.hpp index cd09dce..2b864a7 100644 --- a/include/CLI/App.hpp +++ b/include/CLI/App.hpp @@ -817,7 +817,7 @@ protected: if(require_subcommand_ < 0 && selected_subcommands_.size() == 0) throw RequiredError("Subcommand required"); - else if(require_subcommand_ > 0 && selected_subcommands_.size() != require_subcommand_) + else if(require_subcommand_ > 0 && selected_subcommands_.size() != (int) require_subcommand_) throw RequiredError(std::to_string(require_subcommand_) + " subcommand(s) required"); // Convert missing (pairs) to extras (string only) diff --git a/tests/AppTest.cpp b/tests/AppTest.cpp index 3e65edf..55702e7 100644 --- a/tests/AppTest.cpp +++ b/tests/AppTest.cpp @@ -518,8 +518,8 @@ TEST_F(TApp, RequiresMultiFlags) { TEST_F(TApp, RequiresMixedFlags) { CLI::Option* opt1 = app.add_flag("--opt1"); - CLI::Option* opt2 = app.add_flag("--opt2"); - CLI::Option* opt3 = app.add_flag("--opt3"); + app.add_flag("--opt2"); + app.add_flag("--opt3"); app.add_flag("--optall")->requires(opt1, "--opt2", "--opt3"); EXPECT_NO_THROW(run());