Fix some bugs and problems in gossip

nodemcu / nodemcu-firmware

Lua based interactive firmware for ESP8266, ESP8285 and ESP32

https://nodemcu.readthedocs.io

MIT License

7.64k stars 3.12k forks source link

Closed HHHartmann closed 2 years ago

HHHartmann commented 2 years ago

Fixes issues in gossip.

[x] This PR is for the dev branch rather than for the release branch.
[x] This PR is compliant with the other contributing guidelines as well (if not, please describe why).
[x] I have thoroughly tested my contribution.
[x] The code changes are reflected in the documentation at docs/*.

During working with gossip I found some bugs and weaknesses.

HHHartmann commented 2 years ago

Please review and/or merge

HHHartmann commented 2 years ago

@alexandruantochi would you care to have a look?

HHHartmann commented 2 years ago

Anyone mind if I merge this?

marcelstoer commented 2 years ago

I'll merge it. It's been up for review for nearly two weeks, the original author didn't even reply...you did what you could.

HHHartmann commented 2 years ago

Thx