[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: