Fix logical not usage error

phord · web-flow · commit fdb14b61fd88 · 2022-02-13T15:25:54.000-08:00
Esp32 compiler has `-Werror=logical-not-parentheses` by default and won't compile original code.
diff --git a/src/HttpClient.cpp b/src/HttpClient.cpp
@@ -84,7 +84,7 @@ int HttpClient::startRequest(const char* aURLPath, const char* aHttpMethod,
     {
         if (iServerName)
         {
-            if (!(iClient->connect(iServerName, iServerPort) > 0))
+            if (!iClient->connect(iServerName, iServerPort))
             {
 #ifdef LOGGING
                 Serial.println("Connection failed");
@@ -94,7 +94,7 @@ int HttpClient::startRequest(const char* aURLPath, const char* aHttpMethod,
         }
         else
         {
-            if (!(iClient->connect(iServerAddress, iServerPort) > 0))
+            if (iClient->connect(iServerAddress, iServerPort))
             {
 #ifdef LOGGING
                 Serial.println("Connection failed");

Original file line number	Diff line number	Diff line change
`@@ -84,7 +84,7 @@ int HttpClient::startRequest(const char* aURLPath, const char* aHttpMethod,`
`84`	`84`	`{`
`85`	`85`	`if (iServerName)`
`86`	`86`	`{`
`87`		`- if (!(iClient->connect(iServerName, iServerPort) > 0))`
	`87`	`+ if (!iClient->connect(iServerName, iServerPort))`
`88`	`88`	`{`
`89`	`89`	`#ifdef LOGGING`
`90`	`90`	`Serial.println("Connection failed");`
`@@ -94,7 +94,7 @@ int HttpClient::startRequest(const char* aURLPath, const char* aHttpMethod,`
`94`	`94`	`}`
`95`	`95`	`else`
`96`	`96`	`{`
`97`		`- if (!(iClient->connect(iServerAddress, iServerPort) > 0))`
	`97`	`+ if (iClient->connect(iServerAddress, iServerPort))`
`98`	`98`	`{`
`99`	`99`	`#ifdef LOGGING`
`100`	`100`	`Serial.println("Connection failed");`