1.10.0 wont compile anymore

[exocode]

Hi there,

today I tried to compile my project.

After a few hours I realized that I updated a few days ago the Espressif 32 to version 1.10.0 (formerly I had 1.9.0)

When I downgrade to 1.9.0 it compiles.

1.10.0 fails:

> Executing task: platformio run <

Processing heltec_wifi_kit_32 (platform: espressif32; board: heltec_wifi_kit_32; framework: arduino)
--------------------------------------------------------------------------------------------------------------
Verbose mode can be enabled via `-v, --verbose` option
Mkdir("/Users/jan/Coding/PlatformIO/Projects/DEMO-ESP32-ARD-OTA-THINX/.pio/build/heltec_wifi_kit_32")
CONFIGURATION: https://docs.platformio.org/page/boards/espressif32/heltec_wifi_kit_32.html
PLATFORM: Espressif 32 1.10.0 > Heltec WiFi Kit 32
HARDWARE: ESP32 240MHz, 320KB RAM, 4MB Flash
DEBUG: Current (esp-prog) External (esp-prog, iot-bus-jtag, jlink, minimodule, olimex-arm-usb-ocd, olimex-arm-usb-ocd-h, olimex-arm-usb-tiny-h, olimex-jtag-tiny, tumpa)
PACKAGES: toolchain-xtensa32 2.50200.80 (5.2.0), framework-arduinoespressif32 2.10003.190916 (1.0.3), tool-esptoolpy 1.20600.0 (2.6.0)
LDF: Library Dependency Finder -> http://bit.ly/configure-pio-ldf
LDF Modes: Finder ~ chain, Compatibility ~ soft
Found 38 compatible libraries
Scanning dependencies...
Dependency Graph
|-- <THiNX32> 2.4.189
|   |-- <ArduinoJson> 5.13.1
|   |-- <DNSServer> 1.1.0
|   |   |-- <WiFi> 1.0
|   |-- <EEPROM> 1.0.3
|   |-- <ESP32httpUpdate> 2.1.145
|   |   |-- <FS> 1.0
|   |   |-- <SPIFFS> 1.0
|   |   |   |-- <FS> 1.0
|   |   |-- <HTTPClient> 1.2
|   |   |   |-- <WiFi> 1.0
|   |   |   |-- <WiFiClientSecure> 1.0
|   |   |   |   |-- <WiFi> 1.0
|   |   |-- <Update> 1.0
|   |   |-- <WiFi> 1.0
|   |-- <ESPmDNS> 1.0
|   |   |-- <WiFi> 1.0
|   |-- <HTTPClient> 1.2
|   |   |-- <WiFi> 1.0
|   |   |-- <WiFiClientSecure> 1.0
|   |   |   |-- <WiFi> 1.0
|   |-- <PubSubClient(imroy)> 1.99.1
|   |   |-- <ArduinoJson> 5.13.1
|   |-- <SPIFFS> 1.0
|   |   |-- <FS> 1.0
|   |-- <WebServer> 1.0
|   |   |-- <WiFi> 1.0
|   |   |-- <FS> 1.0
|   |-- <WiFi> 1.0
|   |-- <WiFiClientSecure> 1.0
|   |   |-- <WiFi> 1.0
|   |-- <WifiManager> 0.99.9
|   |   |-- <DNSServer> 1.1.0
|   |   |   |-- <WiFi> 1.0
|   |   |-- <WebServer> 1.0
|   |   |   |-- <WiFi> 1.0
|   |   |   |-- <FS> 1.0
|   |   |-- <WiFi> 1.0
Compiling .pio/build/heltec_wifi_kit_32/src/main.cpp.o
Generating partitions .pio/build/heltec_wifi_kit_32/partitions.bin
Archiving .pio/build/heltec_wifi_kit_32/lib69a/libArduinoJson.a
Indexing .pio/build/heltec_wifi_kit_32/lib69a/libArduinoJson.a
Compiling .pio/build/heltec_wifi_kit_32/lib4c1/WiFi/ETH.cpp.o
Compiling .pio/build/heltec_wifi_kit_32/lib4c1/WiFi/WiFi.cpp.o
Compiling .pio/build/heltec_wifi_kit_32/lib4c1/WiFi/WiFiAP.cpp.o
Compiling .pio/build/heltec_wifi_kit_32/lib4c1/WiFi/WiFiClient.cpp.o
Compiling .pio/build/heltec_wifi_kit_32/lib4c1/WiFi/WiFiGeneric.cpp.o
Compiling .pio/build/heltec_wifi_kit_32/lib4c1/WiFi/WiFiMulti.cpp.o
Compiling .pio/build/heltec_wifi_kit_32/lib4c1/WiFi/WiFiSTA.cpp.o
Compiling .pio/build/heltec_wifi_kit_32/lib4c1/WiFi/WiFiScan.cpp.o
Compiling .pio/build/heltec_wifi_kit_32/lib4c1/WiFi/WiFiServer.cpp.o
Compiling .pio/build/heltec_wifi_kit_32/lib4c1/WiFi/WiFiUdp.cpp.o
Compiling .pio/build/heltec_wifi_kit_32/lib182/DNSServer-esp32/DNSServer.cpp.o
Compiling .pio/build/heltec_wifi_kit_32/lib512/EEPROM/EEPROM.cpp.o
Compiling .pio/build/heltec_wifi_kit_32/lib2eb/FS/FS.cpp.o
Compiling .pio/build/heltec_wifi_kit_32/lib2eb/FS/vfs_api.cpp.o
Compiling .pio/build/heltec_wifi_kit_32/lib9d9/SPIFFS/SPIFFS.cpp.o
/Users/jan/.platformio/packages/framework-arduinoespressif32/libraries/EEPROM/src/EEPROM.cpp: In constructor 'EEPROMClass::EEPROMClass()':
/Users/jan/.platformio/packages/framework-arduinoespressif32/libraries/EEPROM/src/EEPROM.cpp:37:25: warning: converting to non-pointer type 'nvs_handle {aka unsigned int}' from NULL [-Wconversion-null]
   , _user_defined_size(0)
                         ^
/Users/jan/.platformio/packages/framework-arduinoespressif32/libraries/EEPROM/src/EEPROM.cpp: In constructor 'EEPROMClass::EEPROMClass(uint32_t)':
/Users/jan/.platformio/packages/framework-arduinoespressif32/libraries/EEPROM/src/EEPROM.cpp:48:25: warning: converting to non-pointer type 'nvs_handle {aka unsigned int}' from NULL [-Wconversion-null]
   , _user_defined_size(0)
                         ^
/Users/jan/.platformio/packages/framework-arduinoespressif32/libraries/EEPROM/src/EEPROM.cpp: In constructor 'EEPROMClass::EEPROMClass(const char*, uint32_t)':
/Users/jan/.platformio/packages/framework-arduinoespressif32/libraries/EEPROM/src/EEPROM.cpp:58:41: warning: converting to non-pointer type 'nvs_handle {aka unsigned int}' from NULL [-Wconversion-null]
   , _user_defined_size(user_defined_size)
                                         ^
Compiling .pio/build/heltec_wifi_kit_32/lib65b/WiFiClientSecure/WiFiClientSecure.cpp.o
Archiving .pio/build/heltec_wifi_kit_32/lib182/libDNSServer-esp32.a
Archiving .pio/build/heltec_wifi_kit_32/lib4c1/libWiFi.a
Indexing .pio/build/heltec_wifi_kit_32/lib182/libDNSServer-esp32.a
Compiling .pio/build/heltec_wifi_kit_32/lib65b/WiFiClientSecure/ssl_client.cpp.o
Indexing .pio/build/heltec_wifi_kit_32/lib4c1/libWiFi.a
Compiling .pio/build/heltec_wifi_kit_32/lib55f/HTTPClient/HTTPClient.cpp.o
Archiving .pio/build/heltec_wifi_kit_32/lib512/libEEPROM.a
Indexing .pio/build/heltec_wifi_kit_32/lib512/libEEPROM.a
Compiling .pio/build/heltec_wifi_kit_32/libdc1/Update/Updater.cpp.o
Compiling .pio/build/heltec_wifi_kit_32/lib7c5/esp32-http-update/ESP32httpUpdate.cpp.o
Compiling .pio/build/heltec_wifi_kit_32/lib96f/ESPmDNS/ESPmDNS.cpp.o
Archiving .pio/build/heltec_wifi_kit_32/lib2eb/libFS.a
Archiving .pio/build/heltec_wifi_kit_32/lib9d9/libSPIFFS.a
Indexing .pio/build/heltec_wifi_kit_32/lib2eb/libFS.a
Indexing .pio/build/heltec_wifi_kit_32/lib9d9/libSPIFFS.a
Compiling .pio/build/heltec_wifi_kit_32/lib6ee/PubSubSclient/MQTT.cpp.o
Compiling .pio/build/heltec_wifi_kit_32/lib6ee/PubSubSclient/PubSubClient.cpp.o
Compiling .pio/build/heltec_wifi_kit_32/lib34c/WebServer-esp32/Parsing.cpp.o
Archiving .pio/build/heltec_wifi_kit_32/lib65b/libWiFiClientSecure.a
Indexing .pio/build/heltec_wifi_kit_32/lib65b/libWiFiClientSecure.a
Compiling .pio/build/heltec_wifi_kit_32/lib34c/WebServer-esp32/WebServer.cpp.o
Archiving .pio/build/heltec_wifi_kit_32/libdc1/libUpdate.a
Indexing .pio/build/heltec_wifi_kit_32/libdc1/libUpdate.a
lib/PubSubSclient/src/MQTT.cpp: In member function 'void MQTT::Message::write_fixed_header(uint8_t*, uint32_t&, uint32_t) const':
lib/PubSubSclient/src/MQTT.cpp:126:12: warning: enumeration value 'None' not handled in switch [-Wswitch]
     switch (_type) {
            ^
lib/PubSubSclient/src/MQTT.cpp:126:12: warning: enumeration value 'CONNECT' not handled in switch [-Wswitch]
lib/PubSubSclient/src/MQTT.cpp:126:12: warning: enumeration value 'CONNACK' not handled in switch [-Wswitch]
lib/PubSubSclient/src/MQTT.cpp:126:12: warning: enumeration value 'PUBACK' not handled in switch [-Wswitch]
lib/PubSubSclient/src/MQTT.cpp:126:12: warning: enumeration value 'PUBREC' not handled in switch [-Wswitch]
lib/PubSubSclient/src/MQTT.cpp:126:12: warning: enumeration value 'PUBCOMP' not handled in switch [-Wswitch]
lib/PubSubSclient/src/MQTT.cpp:126:12: warning: enumeration value 'SUBACK' not handled in switch [-Wswitch]
lib/PubSubSclient/src/MQTT.cpp:126:12: warning: enumeration value 'UNSUBACK' not handled in switch [-Wswitch]
lib/PubSubSclient/src/MQTT.cpp:126:12: warning: enumeration value 'PINGREQ' not handled in switch [-Wswitch]
lib/PubSubSclient/src/MQTT.cpp:126:12: warning: enumeration value 'PINGRESP' not handled in switch [-Wswitch]
lib/PubSubSclient/src/MQTT.cpp:126:12: warning: enumeration value 'DISCONNECT' not handled in switch [-Wswitch]
lib/PubSubSclient/src/MQTT.cpp:126:12: warning: enumeration value 'Reserved' not handled in switch [-Wswitch]
lib/PubSubSclient/src/MQTT.cpp: In member function 'MQTT::Message* MQTT::PacketParser::parse()':
lib/PubSubSclient/src/MQTT.cpp:322:14: warning: enumeration value 'HaveObject' not handled in switch [-Wswitch]
       switch (_state) {
              ^
In file included from lib/PubSubSclient/src/MQTT.cpp:21:0:
lib/PubSubSclient/src/MQTT.h: In constructor 'MQTT::Publish::Publish(String, const __FlashStringHelper*)':
lib/PubSubSclient/src/MQTT.h:272:14: error: 'MQTT::Publish::_payload_len' will be initialized after [-Werror=reorder]
     uint32_t _payload_len;
              ^
lib/PubSubSclient/src/MQTT.h:271:14: error:   'uint8_t* MQTT::Publish::_payload' [-Werror=reorder]
     uint8_t *_payload;
              ^
lib/PubSubSclient/src/MQTT.cpp:486:3: error:   when initialized here [-Werror=reorder]
   Publish::Publish(String topic, const __FlashStringHelper* payload) :
   ^
In file included from lib/PubSubSclient/src/MQTT.cpp:21:0:
lib/PubSubSclient/src/MQTT.h: In constructor 'MQTT::Publish::Publish(String, MQTT::payload_callback_t, uint32_t)':
Compiling .pio/build/heltec_wifi_kit_32/libfbd/WiFiManager/WiFiManager.cpp.o
lib/PubSubSclient/src/MQTT.h:272:14: error: 'MQTT::Publish::_payload_len' will be initialized after [-Werror=reorder]
     uint32_t _payload_len;
              ^
lib/PubSubSclient/src/MQTT.h:271:14: error:   'uint8_t* MQTT::Publish::_payload' [-Werror=reorder]
     uint8_t *_payload;
              ^
lib/PubSubSclient/src/MQTT.cpp:519:3: error:   when initialized here [-Werror=reorder]
   Publish::Publish(String topic, payload_callback_t pcb, uint32_t length) :
   ^
In file included from lib/PubSubSclient/src/PubSubClient.cpp:7:0:
lib/PubSubSclient/src/PubSubClient.h: In constructor 'PubSubClient::PubSubClient(Client&, IPAddress&, uint16_t)':
lib/PubSubSclient/src/PubSubClient.h:41:9: error: 'PubSubClient::isSubAckFound' will be initialized after [-Werror=reorder]
    bool isSubAckFound;
         ^
lib/PubSubSclient/src/PubSubClient.h:29:14: error:   'IPAddress PubSubClient::server_ip' [-Werror=reorder]
    IPAddress server_ip;
              ^
lib/PubSubSclient/src/PubSubClient.cpp:18:1: error:   when initialized here [-Werror=reorder]
 PubSubClient::PubSubClient(Client& c, IPAddress &ip, uint16_t port) :
 ^
In file included from lib/PubSubSclient/src/PubSubClient.cpp:7:0:
lib/PubSubSclient/src/PubSubClient.h: In constructor 'PubSubClient::PubSubClient(Client&, String, uint16_t)':
lib/PubSubSclient/src/PubSubClient.h:41:9: error: 'PubSubClient::isSubAckFound' will be initialized after [-Werror=reorder]
    bool isSubAckFound;
         ^
lib/PubSubSclient/src/PubSubClient.h:31:13: error:   'uint16_t PubSubClient::server_port' [-Werror=reorder]
    uint16_t server_port;
             ^
lib/PubSubSclient/src/PubSubClient.cpp:28:1: error:   when initialized here [-Werror=reorder]
 PubSubClient::PubSubClient(Client& c, String hostname, uint16_t port) :
 ^
In file included from lib/PubSubSclient/src/PubSubClient.cpp:7:0:
lib/PubSubSclient/src/PubSubClient.h:31:13: error: 'PubSubClient::server_port' will be initialized after [-Werror=reorder]
    uint16_t server_port;
             ^
lib/PubSubSclient/src/PubSubClient.h:30:11: error:   'String PubSubClient::server_hostname' [-Werror=reorder]
    String server_hostname;
           ^
lib/PubSubSclient/src/PubSubClient.cpp:28:1: error:   when initialized here [-Werror=reorder]
 PubSubClient::PubSubClient(Client& c, String hostname, uint16_t port) :
 ^
lib/PubSubSclient/src/PubSubClient.cpp: In member function 'void PubSubClient::_process_message(MQTT::Message*)':
lib/PubSubSclient/src/PubSubClient.cpp:104:10: warning: enumeration value 'None' not handled in switch [-Wswitch]
   switch (msg->type()) {
          ^
lib/PubSubSclient/src/PubSubClient.cpp:104:10: warning: enumeration value 'CONNECT' not handled in switch [-Wswitch]
lib/PubSubSclient/src/PubSubClient.cpp:104:10: warning: enumeration value 'CONNACK' not handled in switch [-Wswitch]
lib/PubSubSclient/src/PubSubClient.cpp:104:10: warning: enumeration value 'PUBACK' not handled in switch [-Wswitch]
lib/PubSubSclient/src/PubSubClient.cpp:104:10: warning: enumeration value 'PUBREC' not handled in switch [-Wswitch]
lib/PubSubSclient/src/PubSubClient.cpp:104:10: warning: enumeration value 'PUBREL' not handled in switch [-Wswitch]
lib/PubSubSclient/src/PubSubClient.cpp:104:10: warning: enumeration value 'PUBCOMP' not handled in switch [-Wswitch]
lib/PubSubSclient/src/PubSubClient.cpp:104:10: warning: enumeration value 'SUBSCRIBE' not handled in switch [-Wswitch]
lib/PubSubSclient/src/PubSubClient.cpp:104:10: warning: enumeration value 'SUBACK' not handled in switch [-Wswitch]
lib/PubSubSclient/src/PubSubClient.cpp:104:10: warning: enumeration value 'UNSUBSCRIBE' not handled in switch [-Wswitch]
lib/PubSubSclient/src/PubSubClient.cpp:104:10: warning: enumeration value 'UNSUBACK' not handled in switch [-Wswitch]
lib/PubSubSclient/src/PubSubClient.cpp:104:10: warning: enumeration value 'DISCONNECT' not handled in switch [-Wswitch]
lib/PubSubSclient/src/PubSubClient.cpp:104:10: warning: enumeration value 'Reserved' not handled in switch [-Wswitch]
lib/PubSubSclient/src/MQTT.cpp: In member function 'virtual MQTT::message_type MQTT::Publish::response_type() const':
lib/PubSubSclient/src/MQTT.cpp:602:3: warning: control reaches end of non-void function [-Wreturn-type]
   }
   ^
Archiving .pio/build/heltec_wifi_kit_32/lib7c5/libesp32-http-update.a
Indexing .pio/build/heltec_wifi_kit_32/lib7c5/libesp32-http-update.a
Archiving .pio/build/heltec_wifi_kit_32/lib55f/libHTTPClient.a
Indexing .pio/build/heltec_wifi_kit_32/lib55f/libHTTPClient.a
cc1plus: some warnings being treated as errors
Compiling .pio/build/heltec_wifi_kit_32/liba45/thinx-firmware-esp32/THiNX32Lib.cpp.o
*** [.pio/build/heltec_wifi_kit_32/lib6ee/PubSubSclient/PubSubClient.cpp.o] Error 1
cc1plus: some warnings being treated as errors
*** [.pio/build/heltec_wifi_kit_32/lib6ee/PubSubSclient/MQTT.cpp.o] Error 1
lib/thinx-firmware-esp32/src/THiNX32Lib.cpp: In member function 'void THiNX::fetchdata()':
lib/thinx-firmware-esp32/src/THiNX32Lib.cpp:457:17: warning: unused variable 'interval' [-Wunused-variable]
   unsigned long interval = 30000;
                 ^
lib/thinx-firmware-esp32/src/THiNX32Lib.cpp:458:17: warning: unused variable 'currentMillis' [-Wunused-variable]
   unsigned long currentMillis = millis(), previousMillis = millis();
                 ^
lib/thinx-firmware-esp32/src/THiNX32Lib.cpp:458:43: warning: unused variable 'previousMillis' [-Wunused-variable]
   unsigned long currentMillis = millis(), previousMillis = millis();
                                           ^
lib/thinx-firmware-esp32/src/THiNX32Lib.cpp: In member function 'void THiNX::fetch_data()':
lib/thinx-firmware-esp32/src/THiNX32Lib.cpp:507:17: warning: unused variable 'interval' [-Wunused-variable]
   unsigned long interval = 30000;
                 ^
lib/thinx-firmware-esp32/src/THiNX32Lib.cpp:508:17: warning: unused variable 'currentMillis' [-Wunused-variable]
   unsigned long currentMillis = millis(), previousMillis = millis();
                 ^
lib/thinx-firmware-esp32/src/THiNX32Lib.cpp:508:43: warning: unused variable 'previousMillis' [-Wunused-variable]
   unsigned long currentMillis = millis(), previousMillis = millis();
                                           ^
lib/thinx-firmware-esp32/src/THiNX32Lib.cpp: In member function 'const char* THiNX::thinx_mac()':
lib/thinx-firmware-esp32/src/THiNX32Lib.cpp:2069:51: warning: format '%X' expects argument of type 'unsigned int', but argument 3 has type 'uint64_t {aka long long unsigned int}' [-Wformat=]
   sprintf(mac_string, "5CCF%6X", ESP.getEfuseMac());
                                                   ^
lib/thinx-firmware-esp32/src/THiNX32Lib.cpp:2069:51: warning: format '%X' expects argument of type 'unsigned int', but argument 3 has type 'uint64_t {aka long long unsigned int}' [-Wformat=]
========================================= [FAILED] Took 9.96 seconds =========================================
The terminal process terminated with exit code: 1

Terminal will be reused by tasks, press any key to close it.

[valeros]

Hi @exocode ! Does it work in Arduino IDE? Could you also try the upstream version of the platform with the latest Arduino package? Thanks!

[exocode]

Hi @valeros, to be honest, I've no clue, nor the Arduino IDE installed/in use (I use only VSCode). I hope someone else is familiar with it. Sorry for not able to help you

[Bmooij]

It looks like there is a problem in the PubSubClient lib.
Which PubSubClient do you use?

[Bmooij]

See: https://stackoverflow.com/questions/1564937/gcc-warning-will-be-initialized-after
The problem is in the initialization order.

For example:

class MyClass {
    private:
        uint8_t number1;
        uint8_t number2;
       
    public:
       MyClass(uint8_t a, uint8_t b) : number2(a), number1(b) {}
}

That throws an error because number2 is initialized before number1.

To fix it you can do 2 things.
Reorder the (private) fields

class MyClass {
    private:
        uint8_t number2;
        uint8_t number1;
       
    public:
       MyClass(uint8_t a, uint8_t b) : number2(a), number1(b) {}
}

or change the initialization in the constructor:

class MyClass {
    private:
        uint8_t number1;
        uint8_t number2;
       
    public:
       MyClass(uint8_t a, uint8_t b) : number1(b), number2(a) {}
}

[Bmooij]

This a result of:
espressif/arduino-esp32#2707

[valeros]

Feel free to reopen the issue if can reproduce the issue with the latest espressif32 platform.