Fixing return unreachable warning on NVCC

Using `else` to make this explicit instead of a lone return.
fmtlib · Jul 9, 2018 · 5f4121e · 5f4121e
1 parent 60c662b
commit 5f4121e
Showing 1 changed file with 11 additions and 11 deletions.
diff --git a/include/fmt/format.h b/include/fmt/format.h
@@ -1476,17 +1476,17 @@ void arg_map<Context>::init(const basic_format_args<Context> &args) {
           break; // Do nothing.
       }
     }
-    return;
-  }
-  for (unsigned i = 0; ; ++i) {
-    switch (args.args_[i].type_) {
-      case internal::none_type:
-        return;
-      case internal::named_arg_type:
-        push_back(args.args_[i].value_);
-        break;
-      default:
-        break; // Do nothing.
+  } else {
+    for (unsigned i = 0;/*nothing*/; ++i) {
+      switch (args.args_[i].type_) {
+        case internal::none_type:
+          return;
+        case internal::named_arg_type:
+          push_back(args.args_[i].value_);
+          break;
+        default:
+          break; // Do nothing.
+      }
     }
   }
 }