Xargon/Duckstation
1
0
Fork 0
mirror of https://github.com/RetroDECK/Duckstation.git synced 2024-11-29 09:05:41 +00:00
Code Issues Actions Packages Projects Releases Wiki Activity

StringUtil: Fix inequal-length EqualNoCase

Browse source
This commit is contained in:
Stenzek 2024-02-04 14:06:58 +10:00
parent 8170475c6b
commit a957fb669d
No known key found for this signature in database
3 changed files with 6 additions and 6 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

6
src/common/string_util.h
View file

@ -63,10 +63,10 @@ static inline int Strncasecmp(const char* s1, const char* s2, std::size_t n)
// Case-insensitive equality of string views.
static inline bool EqualNoCase(std::string_view s1, std::string_view s2)
{
if (s1.empty() || s2.empty())
return (s1.empty() == s2.empty());
if (s1.length() != s2.length())
return false;
return (Strncasecmp(s1.data(), s2.data(), std::min(s1.length(), s2.length())) == 0);
return (Strncasecmp(s1.data(), s2.data(), s1.length()) == 0);
}
/// Wrapper around std::from_chars

4
src/core/settings.cpp
View file

@ -1358,12 +1358,12 @@ static constexpr const std::array s_controller_display_names = {
TRANSLATE_NOOP("ControllerType", "PlayStation Mouse"),
TRANSLATE_NOOP("ControllerType", "NeGcon")};
std::optional<ControllerType> Settings::ParseControllerTypeName(const char* str)
std::optional<ControllerType> Settings::ParseControllerTypeName(std::string_view str)
{
int index = 0;
for (const char* name : s_controller_type_names)
{
if (StringUtil::Strcasecmp(name, str) == 0)
if (StringUtil::EqualNoCase(str, name))
return static_cast<ControllerType>(index);
index++;

2
src/core/settings.h
View file

@ -409,7 +409,7 @@ struct Settings
static const char* GetAudioBackendName(AudioBackend backend);
static const char* GetAudioBackendDisplayName(AudioBackend backend);
static std::optional<ControllerType> ParseControllerTypeName(const char* str);
static std::optional<ControllerType> ParseControllerTypeName(std::string_view str);
static const char* GetControllerTypeName(ControllerType type);
static const char* GetControllerTypeDisplayName(ControllerType type);