Fixing a few compiler warnings to allow compilation with -Wall -Wextr…

…a and -Werror. (#3153)
esp8266 · May 8, 2017 · 2d9e767 · 2d9e767
1 parent db4d3e0
commit 2d9e767
Show file tree

Hide file tree

Showing 3 changed files with 2 additions and 3 deletions.
diff --git a/libraries/ESP8266HTTPClient/src/ESP8266HTTPClient.cpp b/libraries/ESP8266HTTPClient/src/ESP8266HTTPClient.cpp
@@ -132,7 +132,6 @@ bool HTTPClient::begin(String url)
 bool HTTPClient::beginInternal(String url, const char* expectedProtocol)
 {
     DEBUG_HTTPCLIENT("[HTTP-Client][begin] url: %s\n", url.c_str());
-    bool hasPort = false;
     clear();
 
     // check for : (http: or https:

diff --git a/libraries/ESP8266WiFi/src/WiFiClientSecure.cpp b/libraries/ESP8266WiFi/src/WiFiClientSecure.cpp
@@ -601,7 +601,7 @@ bool WiFiClientSecure::loadPrivateKey(Stream& stream, size_t size)
 extern "C" int __ax_port_read(int fd, uint8_t* buffer, size_t count)
 {
     ClientContext* _client = SSLContext::getIOContext(fd);
-    if (!_client || _client->state() != ESTABLISHED && !_client->getSize()) {
+    if (!_client || (_client->state() != ESTABLISHED && !_client->getSize())) {
         errno = EIO;
         return -1;
     }

diff --git a/tools/sdk/lwip/include/lwip/debug.h b/tools/sdk/lwip/include/lwip/debug.h
@@ -63,7 +63,7 @@
 
 #ifndef LWIP_NOASSERT
 #define LWIP_ASSERT(message, assertion) do { if(!(assertion)) \
-  LWIP_PLATFORM_ASSERT(message); } while(0)
+	{ LWIP_PLATFORM_ASSERT(message); } } while(0)
 #else  /* LWIP_NOASSERT */
 #define LWIP_ASSERT(message, assertion) 
 #endif /* LWIP_NOASSERT */