Exverge/suyu
3
0
Fork 0
forked from suyu/suyu
Code Pull requests Activity

settings_setting: Fix errors

Browse source 
ToString didn't have a constexpr if statement where needed.
Canonicalize missed an else, causing unreachable code error on MSVC.
This commit is contained in:
lat9nq 2023-06-11 00:41:14 -04:00
parent 04d4b6ab80
commit 4903f40efe
1 changed files with 3 additions and 2 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

5
src/common/settings_setting.h
View file

@ -142,7 +142,7 @@ protected:
return value_.has_value() ? std::to_string(*value_) : "none";
} else if constexpr (std::is_same<Type, bool>()) {
return value_ ? "true" : "false";
} else if (std::is_same<Type, AudioEngine>()) {
} else if constexpr (std::is_same<Type, AudioEngine>()) {
return CanonicalizeEnum(value_);
} else {
return std::to_string(static_cast<u64>(value_));
@ -222,8 +222,9 @@ public:
[[nodiscard]] std::string constexpr Canonicalize() const override {
if constexpr (std::is_enum<Type>::value) {
return CanonicalizeEnum(this->GetValue());
} else {
return ToString(this->GetValue());
}
return ToString(this->GetValue());
}
/**