From 1e4bf14a3eef4894f65783b4940d98777d5e2263 Mon Sep 17 00:00:00 2001 From: Junxiao Shi Date: Mon, 17 Sep 2018 15:33:01 -0400 Subject: [PATCH] cores: replace max, min, round macros with imports from std (#1783) fixes #1734 --- cores/esp32/Arduino.h | 16 +++++++++------- libraries/WiFi/src/WiFiClient.h | 2 -- libraries/WiFi/src/WiFiGeneric.cpp | 9 +++------ libraries/WiFi/src/WiFiMulti.h | 2 -- 4 files changed, 12 insertions(+), 17 deletions(-) diff --git a/cores/esp32/Arduino.h b/cores/esp32/Arduino.h index d60d3fe0..98e0176f 100644 --- a/cores/esp32/Arduino.h +++ b/cores/esp32/Arduino.h @@ -75,7 +75,6 @@ #define abs(x) ((x)>0?(x):-(x)) #define constrain(amt,low,high) ((amt)<(low)?(low):((amt)>(high)?(high):(amt))) -#define round(x) ((x)>=0?(long)((x)+0.5):(long)((x)-0.5)) #define radians(deg) ((deg)*DEG_TO_RAD) #define degrees(rad) ((rad)*RAD_TO_DEG) #define sq(x) ((x)*(x)) @@ -146,6 +145,9 @@ void shiftOut(uint8_t dataPin, uint8_t clockPin, uint8_t bitOrder, uint8_t val); #ifdef __cplusplus } +#include +#include + #include "WCharacter.h" #include "WString.h" #include "Stream.h" @@ -158,6 +160,12 @@ void shiftOut(uint8_t dataPin, uint8_t clockPin, uint8_t bitOrder, uint8_t val); #include "HardwareSerial.h" #include "Esp.h" +using std::isinf; +using std::isnan; +using std::max; +using std::min; +using ::round; + uint16_t makeWord(uint16_t w); uint16_t makeWord(byte h, byte l); @@ -176,12 +184,6 @@ extern "C" void configTzTime(const char* tz, long random(long); #endif /* __cplusplus */ -#ifndef _GLIBCXX_VECTOR -// arduino is not compatible with std::vector -#define min(a,b) ((a)<(b)?(a):(b)) -#define max(a,b) ((a)>(b)?(a):(b)) -#endif - #define _min(a,b) ((a)<(b)?(a):(b)) #define _max(a,b) ((a)>(b)?(a):(b)) diff --git a/libraries/WiFi/src/WiFiClient.h b/libraries/WiFi/src/WiFiClient.h index 78f3fd94..e87708f5 100644 --- a/libraries/WiFi/src/WiFiClient.h +++ b/libraries/WiFi/src/WiFiClient.h @@ -23,8 +23,6 @@ #include "Arduino.h" #include "Client.h" -#undef min -#undef max #include class WiFiClientSocketHandle; diff --git a/libraries/WiFi/src/WiFiGeneric.cpp b/libraries/WiFi/src/WiFiGeneric.cpp index f4d86875..f26ca615 100644 --- a/libraries/WiFi/src/WiFiGeneric.cpp +++ b/libraries/WiFi/src/WiFiGeneric.cpp @@ -46,9 +46,6 @@ extern "C" { } //extern "C" #include "esp32-hal-log.h" - -#undef min -#undef max #include #include "sdkconfig.h" @@ -156,8 +153,8 @@ static bool espWiFiStart(bool persistent){ return false; } _esp_wifi_started = true; - system_event_t event; - event.event_id = SYSTEM_EVENT_WIFI_READY; + system_event_t event; + event.event_id = SYSTEM_EVENT_WIFI_READY; WiFiGenericClass::_eventCallback(nullptr, &event); return true; @@ -496,7 +493,7 @@ bool WiFiGenericClass::mode(wifi_mode_t m) } else if(cm && !m){ return espWiFiStop(); } - + esp_err_t err; err = esp_wifi_set_mode(m); if(err){ diff --git a/libraries/WiFi/src/WiFiMulti.h b/libraries/WiFi/src/WiFiMulti.h index 140f6386..d628c444 100644 --- a/libraries/WiFi/src/WiFiMulti.h +++ b/libraries/WiFi/src/WiFiMulti.h @@ -27,8 +27,6 @@ #define WIFICLIENTMULTI_H_ #include "WiFi.h" -#undef min -#undef max #include typedef struct {