mirrors/CLI11
1
0
Fork 0
mirror of https://github.com/CLIUtils/CLI11.git synced 2025-04-30 20:53:52 +00:00
Code Issues Releases Wiki Activity

Resolve two switch related warnings on GCC 8 (#255)

Browse Source 
* Add default case to resolve -Werror=switch-default.

* Add missing enumeration values to resolve -Werror=switch-enum.

* Fix coverage
This commit is contained in:
Andrew Hardin 2019-03-22 17:48:00 -06:00 committed by Henry Schreiner
parent 80420d62e2
commit 6be9f0c659
1 changed files with 9 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

9
include/CLI/App.hpp
View File

@ -2234,6 +2234,12 @@ class App {
if(retval && positionals_at_end_) { if(retval && positionals_at_end_) {
positional_only = true; positional_only = true;
} }
break;
// LCOV_EXCL_START
default:
HorribleError("unrecognized classifier (you should not see this!)");
// LCOV_EXCL_END
} }
return retval; return retval;
} }
@ -2396,6 +2402,9 @@ class App {
if(!detail::split_windows_style(current, arg_name, value)) if(!detail::split_windows_style(current, arg_name, value))
throw HorribleError("windows option parsed but missing! You should not see this"); throw HorribleError("windows option parsed but missing! You should not see this");
break; break;
case detail::Classifier::SUBCOMMAND:
case detail::Classifier::POSITIONAL_MARK:
case detail::Classifier::NONE:
default: default:
throw HorribleError("parsing got called with invalid option! You should not see this"); throw HorribleError("parsing got called with invalid option! You should not see this");
} }