From db1a419f5c9326cce906184667c4b880ff91bf1e Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Trygve=20Laugst=C3=B8l?= Date: Thu, 20 Apr 2017 09:38:58 +0200 Subject: [PATCH] Fixing a few compiler warnings to allow compilation with -Wall -Wextra and -Werror. --- libraries/ESP8266HTTPClient/src/ESP8266HTTPClient.cpp | 1 - libraries/ESP8266WiFi/src/WiFiClientSecure.cpp | 2 +- tools/sdk/lwip/include/lwip/debug.h | 2 +- 3 files changed, 2 insertions(+), 3 deletions(-) diff --git a/libraries/ESP8266HTTPClient/src/ESP8266HTTPClient.cpp b/libraries/ESP8266HTTPClient/src/ESP8266HTTPClient.cpp index 49c1bbca30..31e86277a7 100644 --- 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 index da7248fab6..04ee5e98f5 100644 --- a/libraries/ESP8266WiFi/src/WiFiClientSecure.cpp +++ b/libraries/ESP8266WiFi/src/WiFiClientSecure.cpp @@ -598,7 +598,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 index d8359ea3a5..1950a3e2eb 100644 --- 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 */