diff options
author | lat9nq <22451773+lat9nq@users.noreply.github.com> | 2023-09-13 19:31:46 +0200 |
---|---|---|
committer | lat9nq <22451773+lat9nq@users.noreply.github.com> | 2023-09-13 19:36:25 +0200 |
commit | 0098ecb6090b767f13683136c28fee97b8b127c7 (patch) | |
tree | 33545276b9a19d2e503ae5effd49cdfa737ecc80 | |
parent | android/config: Remove uncaught usage of stoul (diff) | |
download | yuzu-0098ecb6090b767f13683136c28fee97b8b127c7.tar yuzu-0098ecb6090b767f13683136c28fee97b8b127c7.tar.gz yuzu-0098ecb6090b767f13683136c28fee97b8b127c7.tar.bz2 yuzu-0098ecb6090b767f13683136c28fee97b8b127c7.tar.lz yuzu-0098ecb6090b767f13683136c28fee97b8b127c7.tar.xz yuzu-0098ecb6090b767f13683136c28fee97b8b127c7.tar.zst yuzu-0098ecb6090b767f13683136c28fee97b8b127c7.zip |
-rw-r--r-- | src/common/settings_common.h | 10 | ||||
-rw-r--r-- | src/common/settings_setting.h | 36 |
2 files changed, 38 insertions, 8 deletions
diff --git a/src/common/settings_common.h b/src/common/settings_common.h index 5b170dfd5..067234aab 100644 --- a/src/common/settings_common.h +++ b/src/common/settings_common.h @@ -225,6 +225,16 @@ public: */ [[nodiscard]] virtual constexpr u32 EnumIndex() const = 0; + /** + * @returns True if the underlying type is a floating point storage + */ + [[nodiscard]] virtual constexpr bool IsFloatingPoint() const = 0; + + /** + * @returns True if the underlying type is a integer storage + */ + [[nodiscard]] virtual constexpr bool IsIntegral() const = 0; + /* * Switchable settings */ diff --git a/src/common/settings_setting.h b/src/common/settings_setting.h index e10843c73..79d2b715f 100644 --- a/src/common/settings_setting.h +++ b/src/common/settings_setting.h @@ -10,6 +10,7 @@ #include <string> #include <typeindex> #include <typeinfo> +#include <fmt/core.h> #include "common/common_types.h" #include "common/settings_common.h" #include "common/settings_enums.h" @@ -112,11 +113,12 @@ protected: return value_.has_value() ? std::to_string(*value_) : "none"; } else if constexpr (std::is_same_v<Type, bool>) { return value_ ? "true" : "false"; - } else if constexpr (std::is_same_v<Type, AudioEngine>) { - // Compatibility with old AudioEngine setting being a string - return CanonicalizeEnum(value_); + } else if constexpr (std::is_floating_point_v<Type>) { + return fmt::format("{:f}", value_); + } else if constexpr (std::is_enum_v<Type>) { + return std::to_string(static_cast<u32>(value_)); } else { - return std::to_string(static_cast<u64>(value_)); + return std::to_string(value_); } } @@ -180,13 +182,15 @@ public: this->SetValue(static_cast<u32>(std::stoul(input))); } else if constexpr (std::is_same_v<Type, bool>) { this->SetValue(input == "true"); - } else if constexpr (std::is_same_v<Type, AudioEngine>) { - this->SetValue(ToEnum<Type>(input)); + } else if constexpr (std::is_same_v<Type, float>) { + this->SetValue(std::stof(input)); } else { this->SetValue(static_cast<Type>(std::stoll(input))); } } catch (std::invalid_argument&) { this->SetValue(this->GetDefault()); + } catch (std::out_of_range&) { + this->SetValue(this->GetDefault()); } } @@ -215,11 +219,27 @@ public: } } + [[nodiscard]] constexpr bool IsFloatingPoint() const final { + return std::is_floating_point_v<Type>; + } + + [[nodiscard]] constexpr bool IsIntegral() const final { + return std::is_integral_v<Type>; + } + [[nodiscard]] std::string MinVal() const override final { - return this->ToString(minimum); + if constexpr (std::is_arithmetic_v<Type> && !ranged) { + return this->ToString(std::numeric_limits<Type>::min()); + } else { + return this->ToString(minimum); + } } [[nodiscard]] std::string MaxVal() const override final { - return this->ToString(maximum); + if constexpr (std::is_arithmetic_v<Type> && !ranged) { + return this->ToString(std::numeric_limits<Type>::max()); + } else { + return this->ToString(maximum); + } } [[nodiscard]] constexpr bool Ranged() const override { |