Skip to content

F
fmt
Commit c9bb5468 authored by Victor Zverovich's avatar Victor Zverovich
Browse files
Options

Fix clang warning

parent b26e76ef
Hide whitespace changes
Inline Side-by-side
Showing with 3 additions and 2 deletions
fmt/format.h
View file @ c9bb5468
...@@ -1514,7 +1514,8 @@ class ArgVisitor { ...@@ -1514,7 +1514,8 @@ class ArgVisitor {
*/ */
Result visit(const Arg &arg) { Result visit(const Arg &arg) {
switch (arg.type) { switch (arg.type) {
default: case Arg::NONE:
case Arg::NAMED_ARG:
FMT_ASSERT(false, "invalid argument type"); FMT_ASSERT(false, "invalid argument type");
return Result(); return Result();
case Arg::INT: case Arg::INT:
......
test/util-test.cc
View file @ c9bb5468
...@@ -708,7 +708,7 @@ TEST(ArgVisitorTest, VisitUnhandledArg) { ...@@ -708,7 +708,7 @@ TEST(ArgVisitorTest, VisitUnhandledArg) {
TEST(ArgVisitorTest, VisitInvalidArg) { TEST(ArgVisitorTest, VisitInvalidArg) {
Arg arg = Arg(); Arg arg = Arg();
arg.type = static_cast<Arg::Type>(Arg::CUSTOM + 1); arg.type = static_cast<Arg::Type>(Arg::NONE);
EXPECT_ASSERT(TestVisitor().visit(arg), "invalid argument type"); EXPECT_ASSERT(TestVisitor().visit(arg), "invalid argument type");
} }
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment