-
Notifications
You must be signed in to change notification settings - Fork 40
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Pub ack #83
Pub ack #83
Conversation
As newer versions refuse to work so far I used the working version from Arduino Uncommenting "#define USE_ETHERNET true" uses EthernetClient/EthernetServer works with W5500
added RP2040, using Ethernet (for example for W5500 EVB pico)
Apply richievos memory leak fix
apply richievos memory leak fix
Brought up to the latest branch (main) and added SubAck packets.
reduces network traffic from unnecessary Dup packets
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I could agree wth this PR, anyway, this kind of change needs some unit tests. I'm on another issue now writing a set of unit tests for issues.
I have altered the PR to include the fix for #84 already. This might save you having to test twice. The fix for #84 was borrowed from this PR anyway, I just had originally kept the bit manipulation out your code as I was not sure if the omission was intentionally. The revised PR has the fix moved up a few lines to take effect in the earlier code too and corrected a mistake I had made (&& instead of &). Instead of using PublishID() it would also be possible to fetch the ID bytes straight from [payload], this would prevent misuse of PublishID() out of context. |
Thans for the PR, I'm closing it but have reflected changes with a unit test in 90435b1 |
Suggestion as per discussion.
It solves duplicate packets