Skip to content

F
fmt
Commit 674999c5 authored by Steven Hoving's avatar Steven Hoving Committed by Victor Zverovich
Browse files
Options

fix vs2017 warning fmt::v5::localtime 'not all control paths return a value'.

parent e4fea22d
Hide whitespace changes
Inline Side-by-side
Showing with 2 additions and 0 deletions
include/fmt/time.h
View file @ 674999c5
...@@ -58,6 +58,7 @@ inline std::tm localtime(std::time_t time) { ...@@ -58,6 +58,7 @@ inline std::tm localtime(std::time_t time) {
return lt.tm_; return lt.tm_;
// Too big time values may be unsupported. // Too big time values may be unsupported.
FMT_THROW(format_error("time_t value out of range")); FMT_THROW(format_error("time_t value out of range"));
return {};
} }
// Thread-safe replacement for std::gmtime // Thread-safe replacement for std::gmtime
...@@ -93,6 +94,7 @@ inline std::tm gmtime(std::time_t time) { ...@@ -93,6 +94,7 @@ inline std::tm gmtime(std::time_t time) {
return gt.tm_; return gt.tm_;
// Too big time values may be unsupported. // Too big time values may be unsupported.
FMT_THROW(format_error("time_t value out of range")); FMT_THROW(format_error("time_t value out of range"));
return {};
} }
namespace internal { namespace internal {
......
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