Sinipelto
/
banan-os
forked from Bananymous/banan-os
1
0
Fork 0
Code Pull Requests Activity

BAN: String and StringView don't have to compute strlen for equality

This commit is contained in:
Bananymous 2022-12-20 11:36:01 +02:00
parent c82caacbaa
commit d5a068f90c
2 changed files with 4 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
BAN/BAN/String.cpp
View File

@ -141,9 +141,9 @@ namespace BAN
bool String::operator==(const char* other) const bool String::operator==(const char* other) const
{ {
if (m_size != strlen(other)) if (memcmp(m_data, other, m_size))
return false; return false;
return memcmp(m_data, other, m_size) == 0; return other[m_size] == '\0';
} }
ErrorOr<void> String::Resize(size_type size, char ch) ErrorOr<void> String::Resize(size_type size, char ch)

4
BAN/BAN/StringView.cpp
View File

@ -45,9 +45,9 @@ namespace BAN
bool StringView::operator==(const char* other) const bool StringView::operator==(const char* other) const
{ {
if (m_size != strlen(other)) if (memcmp(m_data, other, m_size))
return false; return false;
return memcmp(m_data, other, m_size) == 0; return other[m_size] == '\0';
} }
StringView StringView::Substring(size_type index, size_type len) const StringView StringView::Substring(size_type index, size_type len) const