seedlingattempt/godot
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Add comments to String::size to lead people to length() and explain the difference.

Browse Source 
# Conflicts:
#	core/string/ustring.h
This commit is contained in:
Lukas Tenbrink
2025-09-26 16:47:15 +02:00
parent d705613db3
commit bac9a9be85
1 changed files with 6 additions and 0 deletions
Download Patch File Download Diff File Expand all files Collapse all files

6
core/string/ustring.h
View File

@ -179,7 +179,10 @@ public:
_FORCE_INLINE_ const T *ptr() const { return _cowdata.ptr(); }
_FORCE_INLINE_ const T *get_data() const { return size() ? ptr() : &_null; }
// Returns the number of characters in the buffer, including the terminating NUL character.
// In most cases, length() should be used instead.
_FORCE_INLINE_ int size() const { return _cowdata.size(); }
// Returns the number of characters in the string (excluding terminating NUL character).
_FORCE_INLINE_ int length() const { return size() ? size() - 1 : 0; }
_FORCE_INLINE_ bool is_empty() const { return length() == 0; }
@ -299,7 +302,10 @@ public:
_FORCE_INLINE_ const char32_t *ptr() const { return _cowdata.ptr(); }
_FORCE_INLINE_ const char32_t *get_data() const { return size() ? ptr() : &_null; }
// Returns the number of characters in the buffer, including the terminating NUL character.
// In most cases, length() should be used instead.
_FORCE_INLINE_ int size() const { return _cowdata.size(); }
// Returns the number of characters in the string (excluding terminating NUL character).
_FORCE_INLINE_ int length() const { return size() ? size() - 1 : 0; }
_FORCE_INLINE_ bool is_empty() const { return length() == 0; }