Merge pull request #80824 from MewPurPur/optimize-left-and-right

akien-mga · akien-mga · commit 970be7afdc11 · 2023-08-21T08:26:15.000+02:00
Optimize `String.left()` and `String.right()`
diff --git a/core/string/ustring.cpp b/core/string/ustring.cpp
@@ -3665,7 +3665,9 @@ String String::left(int p_len) const {
 		return *this;
 	}
 
-	return substr(0, p_len);
+	String s;
+	s.copy_from_unchecked(&get_data()[0], p_len);
+	return s;
 }
 
 String String::right(int p_len) const {
@@ -3681,7 +3683,9 @@ String String::right(int p_len) const {
 		return *this;
 	}
 
-	return substr(length() - p_len);
+	String s;
+	s.copy_from_unchecked(&get_data()[length() - p_len], p_len);
+	return s;
 }
 
 char32_t String::unicode_at(int p_idx) const {

Original file line number	Diff line number	Diff line change
`@@ -3665,7 +3665,9 @@ String String::left(int p_len) const {`
`3665`	`3665`	`return *this;`
`3666`	`3666`	`}`
`3667`	`3667`
`3668`		`- return substr(0, p_len);`
	`3668`	`+ String s;`
	`3669`	`+ s.copy_from_unchecked(&get_data()[0], p_len);`
	`3670`	`+ return s;`
`3669`	`3671`	`}`
`3670`	`3672`
`3671`	`3673`	`String String::right(int p_len) const {`
`@@ -3681,7 +3683,9 @@ String String::right(int p_len) const {`
`3681`	`3683`	`return *this;`
`3682`	`3684`	`}`
`3683`	`3685`
`3684`		`- return substr(length() - p_len);`
	`3686`	`+ String s;`
	`3687`	`+ s.copy_from_unchecked(&get_data()[length() - p_len], p_len);`
	`3688`	`+ return s;`
`3685`	`3689`	`}`
`3686`	`3690`
`3687`	`3691`	`char32_t String::unicode_at(int p_idx) const {`