In String::substr_index the end_index now isn't inclusive.

This commit is contained in:
Relintai 2022-04-26 23:03:43 +02:00
parent cd548cc83d
commit 131d4b49cc
2 changed files with 14 additions and 18 deletions

View File

@ -253,17 +253,13 @@ String String::substr(const int start_index, const int len) const {
}
String String::substr_index(const int start_index, const int end_index) const {
if (start_index == end_index) {
return String();
}
ERR_FAIL_INDEX_V(start_index, _size, String());
ERR_FAIL_INDEX_V(end_index, _size, String());
ERR_FAIL_INDEX_V(start_index, _size + 1, String());
ERR_FAIL_INDEX_V(end_index, _size + 1, String());
ERR_FAIL_COND_V(start_index > end_index, String());
String str;
str.ensure_capacity(end_index - start_index + 1);
for (int i = start_index; i <= end_index; ++i) {
for (int i = start_index; i < end_index; ++i) {
str._data[str._size++] = _data[i];
}
@ -600,12 +596,12 @@ String String::get_slice(const char splitter, int index) {
}
if (count == index + 1) {
return substr_index(start_index, i - 1);
return substr_index(start_index, i);
}
}
}
return substr_index(start_index, _size - 1);
return substr_index(start_index, _size);
}
String String::get_slice(const String &splitter, int index) {
if (_size == 0) {
@ -626,11 +622,11 @@ String String::get_slice(const String &splitter, int index) {
}
if (count == index + 1) {
return substr_index(start_index, n - 1);
return substr_index(start_index, n);
}
}
return substr_index(start_index, _size - 1);
return substr_index(start_index, _size);
}
Vector<String> String::split(const char splitter) const {
@ -648,7 +644,7 @@ Vector<String> String::split(const char splitter) const {
if (start_index == i) {
v.push_back(String());
} else {
v.push_back(substr_index(start_index, i - 1));
v.push_back(substr_index(start_index, i));
}
start_index = i + 1;
@ -656,7 +652,7 @@ Vector<String> String::split(const char splitter) const {
}
if (start_index < _size - 1) {
v.push_back(substr_index(start_index, _size - 1));
v.push_back(substr_index(start_index, _size));
}
return v;
@ -674,13 +670,13 @@ Vector<String> String::split(const String &splitter) const {
while (n != -1) {
n = find(splitter, n);
v.push_back(substr_index(start_index, n - 1));
v.push_back(substr_index(start_index, n));
start_index = n + splitter.size();
}
if (start_index < _size - 1) {
v.push_back(substr_index(start_index, _size - 1));
v.push_back(substr_index(start_index, _size));
}
return v;
@ -1142,7 +1138,7 @@ String String::path_get_basename() const {
++ssind;
return substr_index(ssind, _size - 1);
return substr_index(ssind, _size);
}
String String::path_get_last_segment() const {
@ -1214,7 +1210,7 @@ String String::file_get_extension() const {
return String();
}
return substr_index(dind + 1, _size - 1);
return substr_index(dind + 1, _size);
}
void String::to_html_special_chars() {

View File

@ -40,7 +40,7 @@ public:
void get_substr(char *into_buf, const int start_index, const int len);
void get_substr_nt(char *into_buf, const int start_index, const int len);
String substr(const int start_index, const int len) const;
String substr_index(const int start_index, const int end_index) const;
String substr_index(const int start_index, const int end_index) const; //end_index is not included
bool contains(const char val) const;
bool contains(const String &val) const;