On 18.07.2019 16:43, Tamas K Lengyel wrote: > --- a/CODING_STYLE > +++ b/CODING_STYLE > @@ -60,8 +60,8 @@ Bracing > ------- > > Braces ('{' and '}') are usually placed on a line of their own, except > -for the do/while loop. This is unlike the Linux coding style and > -unlike K&R. do/while loops are an exception. e.g.: > +for the while-part of do/while loops. This is unlike the Linux coding style > +and unlike K&R. do/while loops are an exception. e.g.: > > if ( condition ) > { > @@ -77,7 +77,8 @@ while ( condition ) > /* Do stuff. */ > } > > -do { > +do > +{ > /* Do stuff. */ > } while ( condition );
I disagree with this change: There's a large number of (correct as per the text prior to your change) instances, and since there's nothing affecting the length of such lines avoiding the extra line is quite desirable imo. Jan _______________________________________________ Xen-devel mailing list Xen-devel@lists.xenproject.org https://lists.xenproject.org/mailman/listinfo/xen-devel