Skip to content

Add platformio configuration; fix compilation warnings

Bertrik Sikken requested to merge bertrik/Metertrekker2MQTT:warnings into master

platformio.ini

Dit is een configuratie file voor platformio, hiermee kan je de code compileren op de command line zonder ArduinoIDE, ook handig als je wil draaien op een continuous integration service zoals travis.

Compileren: pio run

Compileren en uploaden: pio run -t upload

Seriele poort monitoren pio device monitor

Libraries en toolchains worden automatisch gedownload en daarna gecached.

De build flag -DMQTT_MAX_PACKET_SIZE=1024 lost als het goed is je probleem op dat je de externe PubSubClient library moet editen.

De CRC16 dependency heb ik niet expliciet toegevoegd, desondanks compilet het toch ... hoe dat zit weet ik nog niet, maar het compilet in ieder geval :)


changes in SMRtoMQTT.ino

This fixes two compile warnings about signed/unsigned comparison. The problem occurs because .length() and .sizeof() don't return an int, but a size_t, so the warning can easily be fixed by changing the loop variable to a size_t too.

Edited by Reinier van der Leer

Merge request reports