From 904793eab427444b60b1bddc5b4b1f9eb143c409 Mon Sep 17 00:00:00 2001 From: Henry Fredrick Schreiner Date: Fri, 24 Feb 2017 17:27:38 -0500 Subject: [PATCH] Fixing a few warnings --- include/CLI/App.hpp | 2 +- tests/AppTest.cpp | 4 ++-- 2 files changed, 3 insertions(+), 3 deletions(-) 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()); -- libgit2 0.21.4