pointless comparison of unsigned integer with zero
This commit is contained in:
parent
c4ee726532
commit
c6c79a0932
@ -1396,7 +1396,8 @@ inline bool isfinite(T) {
|
|||||||
// Converts value to Int and checks that it's in the range [0, upper).
|
// Converts value to Int and checks that it's in the range [0, upper).
|
||||||
template <typename T, typename Int, FMT_ENABLE_IF(std::is_integral<T>::value)>
|
template <typename T, typename Int, FMT_ENABLE_IF(std::is_integral<T>::value)>
|
||||||
inline Int to_nonnegative_int(T value, Int upper) {
|
inline Int to_nonnegative_int(T value, Int upper) {
|
||||||
FMT_ASSERT(value >= 0 && to_unsigned(value) <= to_unsigned(upper),
|
FMT_ASSERT(std::is_unsigned<Int>::value ||
|
||||||
|
(value >= 0 && to_unsigned(value) <= to_unsigned(upper)),
|
||||||
"invalid value");
|
"invalid value");
|
||||||
(void)upper;
|
(void)upper;
|
||||||
return static_cast<Int>(value);
|
return static_cast<Int>(value);
|
||||||
|
|||||||
@ -402,7 +402,7 @@ template <typename T> auto convert_for_visit(T) -> monostate { return {}; }
|
|||||||
template <typename Int>
|
template <typename Int>
|
||||||
FMT_CONSTEXPR auto to_unsigned(Int value) ->
|
FMT_CONSTEXPR auto to_unsigned(Int value) ->
|
||||||
typename std::make_unsigned<Int>::type {
|
typename std::make_unsigned<Int>::type {
|
||||||
FMT_ASSERT(value >= 0, "negative value");
|
FMT_ASSERT(std::is_unsigned<Int>::value || value >= 0, "negative value");
|
||||||
return static_cast<typename std::make_unsigned<Int>::type>(value);
|
return static_cast<typename std::make_unsigned<Int>::type>(value);
|
||||||
}
|
}
|
||||||
|
|
||||||
|
|||||||
Loading…
Reference in New Issue
Block a user