- Updated vector.push_back to .emplace_back to avoid unnecessary copies.

- Updated int port to uint16_t

Author: piscodev

src/R4HttpClient.cpp

@@ -28,7 +28,7 @@ void R4HttpClient::begin(const WiFiSSLClient &sslClient, const String &url)
  * @params: sslClient, url, nport
  * @Description: Initializes client, url, port. Then, extracts the url to get the host, then the endpoint in which where it will do the post request
  */
-void R4HttpClient::begin(const WiFiSSLClient &sslClient, const String &url, const int &nport)
+void R4HttpClient::begin(const WiFiSSLClient &sslClient, const String &url, const uint16_t &nport)
 {
   this->client = sslClient;
   this->setPort(nport);
@@ -55,7 +55,7 @@ void R4HttpClient::extractUrlComponents(const String &url)
   this->endpoint = (endpointIndex == -1) ? "/" : url.substring(endpointIndex);
 }
 
-void R4HttpClient::setPort(const int &nport)
+void R4HttpClient::setPort(const uint16_t &nport)
 {
   this->port = nport;
 }

src/R4HttpClient.h

@@ -109,7 +109,7 @@ enum ResponseState
 class R4HttpClient
 {
   private:
-    int port;
+    uint16_t port;
     String host;
     String endpoint;
     String body;
@@ -118,7 +118,7 @@ class R4HttpClient
     std::vector<String> headers;
 
     void extractUrlComponents(const String &url);
-    void setPort(const int &nport);
+    void setPort(const uint16_t &nport);
     int sendRequest(const String &method, const String &requestBody);
 
     // disable copy constructor and assignment operator
@@ -130,7 +130,7 @@ class R4HttpClient
     ~R4HttpClient();
 
     void begin(const WiFiSSLClient &sslClient, const String &url);
-    void begin(const WiFiSSLClient &sslClient, const String &url, const int &nport);
+    void begin(const WiFiSSLClient &sslClient, const String &url, const uint16_t &nport);
     void addHeader(const String &content);
     void setTimeout(const int &ms);
     void setDebug(const bool &debug);