-
Notifications
You must be signed in to change notification settings - Fork 686
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
Rework FoE Busy handling and enables in global test of win32 #627
base: master
Are you sure you want to change the base?
Conversation
Does this PR supersede the PRs provided by @blocksebastian? |
@lehned , ping |
Yes, that was my old account (which is taken by project paperwork). |
we can close the old #56. I splitted the old one into multiple as this project didn't react and hoped it will be integrated if I make the pull requests smaller and one topic a pull-request. |
The sendpacket counter will become corrupt, in case the client application verifies the packet counter. |
There are two commits:
1st: FoE has incorrect BUSY handling without resending a packet and therefore aborting due timeout on wait of a never sended packet.
2nd: cleanup win32 examples and adding CMakeFiles, additional enabling win32 them by default for tests