[haiku-commits] Re: haiku: hrev51168 - src/kits/shared
- From: Dario Casalinuovo <b.vitruvio@xxxxxxxxx>
- To: haiku-commits@xxxxxxxxxxxxx
- Date: Tue, 16 May 2017 02:46:37 +0200
Hi Axel,
default:
+ {
jsonParseContext.Listener()->HandleError(result,
-1,
"io related read error");
return false;
+ }
BTW, while it doesn't hurt, Dario got that wrong; it's not part of our
coding style guide to always put case blocks in parenthesis.
I know there's a little level of misunderstanding, but I've always referred
explicitly to multi-line statements where there were more than one
statement between case and break.
On the other hand, in the style guide there's an explicit example about
that :)
I just pointed out violations because I've seen there were multiple, and by
experience I know one could just be unaware of that.
Bye,
Dario
Other related posts: