switch case brackets #13
cterwilliger
started this conversation in
General
Replies: 1 comment 1 reply
-
The VeDirectFrameHandler code is from Victron's VEDirect protocol FAQ: (https://www.victronenergy.com/live/vedirect_protocol:faq) They wrote that part and I didn't catch it. Doesn't seem like it should break anything to remove them. What problems did you see? |
Beta Was this translation helpful? Give feedback.
1 reply
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
-
From email:
I'm using your https://github.com/cterwilliger/VeDirectFrameHandler lib.
Thank you for coding this.
It took some time to understand everything, but there is one question open.
Why did you set brackets in VeDirectFrameHandler.cpp
case CHECKSUM:
{
.......
break;
}
As far as I know, in 'switch case' no brackets are needed.
But I tried to delete them and some time later(!!!!) I had huge problems!
I took a long time to find the problem - the missing brackets ;)
So please, can you tell me, what's the secret :)
Thank you,
andi
Beta Was this translation helpful? Give feedback.
All reactions