Use String::length(), even though in this case size() works too.

Relintai · Mar 10, 2024 · 75e8f68 · 75e8f68
1 parent 27d97cf
commit 75e8f68
Showing 1 changed file with 2 additions and 4 deletions.
diff --git a/modules/http_server_simple/http_parser.cpp b/modules/http_server_simple/http_parser.cpp
@@ -194,8 +194,7 @@ void HTTPParser::_process_multipart_header_value(const String &val) {
 
 				if (_multipart_form_name.length() >= 2 && _multipart_form_name.begins_with("\"") && _multipart_form_name.ends_with("\"")) {
 					_multipart_form_name.remove(0);
-					//TODO check if this should be _multipart_form_name.remove(_multipart_form_name.length() - 1);
-					_multipart_form_name.remove(_multipart_form_name.size() - 1);
+					_multipart_form_name.remove(_multipart_form_name.length() - 1);
 				}
 			} else if (kk == "filename") {
 				_multipart_form_filename = vs.get_slicec('=', 1);
@@ -204,8 +203,7 @@ void HTTPParser::_process_multipart_header_value(const String &val) {
 
 				if (_multipart_form_name.length() >= 2 && _multipart_form_name.begins_with("\"") && _multipart_form_name.ends_with("\"")) {
 					_multipart_form_name.remove(0);
-					//TODO check if this should be _multipart_form_name.remove(_multipart_form_name.length() - 1);
-					_multipart_form_name.remove(_multipart_form_name.size() - 1);
+					_multipart_form_name.remove(_multipart_form_name.length() - 1);
 				}
 
 				if (upload_file_store_type == WebServerSimple::FILE_UPLOAD_STORE_TYPE_TEMP_FILES) {