From 9a5afdfc741a35ec18a71295c508c1142391ec09 Mon Sep 17 00:00:00 2001
From: Vaibhav Patel <vai.pateln@gmail.com>
Date: Fri, 26 Feb 2021 05:01:05 -0500
Subject: [PATCH] Fix -Wlogical-not-parentheses warning (#79)

* Fix -Wlogical-not-parentheses warning

The warning occurs because `operator!()` has higher precedence than `operator>()`.

Alternatively, we can use:
```
iClient->connect(iServerAddress, iServerPort) <= 0
```
---
 src/HttpClient.cpp | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/src/HttpClient.cpp b/src/HttpClient.cpp
index 7517eea..4f3410f 100644
--- 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) > 0))
             {
 #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) > 0))
             {
 #ifdef LOGGING
                 Serial.println("Connection failed");