Skip to content

New : function to clear message queue#333

Merged
Miq1 merged 3 commits into
eModbus:masterfrom
ameisso:master
Jan 26, 2024
Merged

New : function to clear message queue#333
Miq1 merged 3 commits into
eModbus:masterfrom
ameisso:master

Conversation

@ameisso

@ameisso ameisso commented Jan 24, 2024

Copy link
Copy Markdown

This is a proposal

In my usage, I sometimes need to send a message with a very high priority (security handbrake).
So clearing the queue helps doing this.

Best

@Miq1

Miq1 commented Jan 24, 2024

Copy link
Copy Markdown
Contributor

I like the idea and your solution looks good. The only thing I would like to add is the same function for the TCP clients as well.

@Miq1 Miq1 left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Wonderful, thank you!

@ameisso

ameisso commented Jan 24, 2024

Copy link
Copy Markdown
Author

Hello,

Thanks for your feedback
I've made the changes. it compiles, but I have nothing to test ATM

Best

@Miq1

Miq1 commented Jan 24, 2024

Copy link
Copy Markdown
Contributor

I will be able to test tomorrow.

@Miq1 Miq1 merged commit 3e21d1f into eModbus:master Jan 26, 2024
@sdrshnptl

Copy link
Copy Markdown

strong feeling that this is a solution to a problem I've encountered recently.
My esp32 on the w5500 modbus server is crashing without cause.
Will validate and update status.
Thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants