UseSpanToReduceAllocations

Author: kiroxas

core/io/http_client_tcp.cpp

@@ -482,7 +482,6 @@ Error HTTPClientTCP::poll() {
 						(rs >= 2 && response_str[rs - 2] == '\n' && response_str[rs - 1] == '\n') ||
 						(rs >= 4 && response_str[rs - 4] == '\r' && response_str[rs - 3] == '\n' && response_str[rs - 2] == '\r' && response_str[rs - 1] == '\n')) {
 					// End of response, parse.
-					response_str.push_back(0);
 					String response = String::utf8((const char *)response_str.ptr(), response_str.size());
 					Vector<String> responses = response.split("\n");
 					body_size = -1;
@@ -506,13 +505,20 @@ Error HTTPClientTCP::poll() {
 						if (s.length() == 0) {
 							continue;
 						}
-						if (s.begins_with("content-length:")) {
-							body_size = s.substr(s.find_char(':') + 1).strip_edges().to_int();
+						const char contentLengthLabel[] = "content-length:";
+						const char transferEncodingLabel[] = "transfer-encoding:";
+						if (s.begins_with(contentLengthLabel)) {
+							const int index = (sizeof(contentLengthLabel) / sizeof(contentLengthLabel[0])) - 1;
+							Span<char32_t> sp = Span<char32_t>(s.ptr() + index, s.length() - index);
+							sp = String::strip_edges_span(sp.begin(), sp.size(), true, true);
+							body_size = String::to_int(sp.begin(), sp.size());
 							body_left = body_size;
 
-						} else if (s.begins_with("transfer-encoding:")) {
-							String encoding = header.substr(header.find_char(':') + 1).strip_edges();
-							if (encoding == "chunked") {
+						} else if (s.begins_with(transferEncodingLabel)) {
+							const int index = (sizeof(transferEncodingLabel) / sizeof(transferEncodingLabel[0])) - 1;
+							Span<char32_t> sp = Span<char32_t>(s.ptr() + index, s.length() - index);
+							sp = String::strip_edges_span(sp.begin(), sp.size(), true, true);
+							if (String("chunked") == sp) {
 								chunked = true;
 							}
 						} else if (s.begins_with("connection: close")) {

core/string/ustring.cpp

@@ -4345,37 +4345,41 @@ String String::dedent() const {
 	return new_string;
 }
 
-String String::strip_edges(bool left, bool right) const {
-	int len = length();
-	int beg = 0, end = len;
+Span<char32_t> String::strip_edges_span(const char32_t *ptr, uint32_t len, bool left, bool right) {
+	int beg = 0, end = static_cast<int>(len);
 
 	if (left) {
-		for (int i = 0; i < len; i++) {
-			if (operator[](i) <= 32) {
+		for (int i = 0; i < end; i++) {
+			if (ptr[i] <= 32) {
 				beg++;
 			} else {
 				break;
 			}
 		}
 	}
 
-	if (right) {
-		for (int i = len - 1; i >= 0; i--) {
-			if (operator[](i) <= 32) {
+	if (right && len) {
+		for (int i = end - 1; i >= beg; i--) {
+			if (ptr[i] <= 32) {
 				end--;
 			} else {
 				break;
 			}
 		}
 	}
 
-	if (beg == 0 && end == len) {
-		return *this;
-	}
+	return Span(ptr + beg, static_cast<uint64_t>(end - beg));
+}
 
-	return substr(beg, end - beg);
+String String::strip_edges(bool left, bool right) const {
+	const char32_t *p = ptr();
+	Span<char32_t> sp = strip_edges_span(p, length(), left, right);
+
+	return substr(sp.begin() - p, sp.size());
 }
 
+
+
 String String::strip_escapes() const {
 	String new_string;
 	for (int i = 0; i < length(); i++) {

core/string/ustring.h

@@ -293,6 +293,8 @@ class String {
 	_FORCE_INLINE_ operator Span<char32_t>() const { return Span(ptr(), length()); }
 	_FORCE_INLINE_ Span<char32_t> span() const { return Span(ptr(), length()); }
 
+	static Span<char32_t> strip_edges_span(const char32_t *sp, uint32_t len, bool left, bool right);
+
 	void remove_at(int p_index) { _cowdata.remove_at(p_index); }
 
 	_FORCE_INLINE_ void clear() { resize(0); }