Fixed comparison of message packet ids

Imroy / pubsubclient

A client library for the ESP8266 that provides support for MQTT

MIT License

434 stars 115 forks source link

Fixed comparison of message packet ids #24

Closed mactro closed 9 years ago

mactro commented 9 years ago

When publishing with QoS 1, publish() always returned false after first 255 messages. It was due to comparision between uint8 and uint16 variables. (line 136). Now publish with QoS1 works fine even after rollover after 2^16 messages.

Imroy commented 9 years ago

Thanks for that. Wow, that was a brain-fart on my behalf. My sensor node only sends a few messages before going back to sleep, so this didn't impact me.