<div dir="ltr"><br><br><div class="gmail_quote"><div dir="ltr">On Mon, Jun 22, 2015 at 2:09 AM Xavier Chantry <<a href="mailto:xavier.chantry@intersec.com">xavier.chantry@intersec.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">ping ;)<br>
<br>
I know these patches are not very exciting, but it would be interesting<br>
for us to have them upstream. And we obviously do not want to change the<br>
default behavior, but add a new option for each item.<br>
<br>
On 22/05/2015 19:04, Xavier Chantry wrote:<br>
> Hello list,<br>
><br>
> During our last internal hackathon at Intersec, we (a team of 4 intersec<br>
> developers with absolutely no knowledge of clang code) tried to adapt<br>
> clang-format to our coding rules.<br>
><br>
> We would like to know if this kind of patches could be of interest, or<br>
> if they are all too specific.<br>
><br>
> Here is a partial list of our patches just to give an idea:<br>
><br>
> 1. Add a space after an assert instruction.<br>
><br>
> assert (ptr);<br>
><br>
> This patch is a bit specific but could be generalized to a list of<br>
> keywords.<br>
><br>
> 2. break line in else if<br>
><br>
> if (foo) {<br>
> } else<br>
> if (bar) {<br>
> }<br>
><br>
> 3. "} else {" on the same line<br>
><br>
> if (foo) {<br>
> } else {<br>
> }<br>
><br>
> 4. align logical operators with if and while<br>
><br>
> if (long_condition1<br>
> &&  long_condition2)<br>
> {<br>
> }<br>
><br>
> Note that the conditions are also aligned.<br>
><br>
> 5. have a regexp instead of a list in IsForEachMacro<br>
><br>
> The idea is that we have a lot of macros matching ".*_for_each_.*" that<br>
> behaves as a for.<br>
><br>
> 6. extra indent or outdent of label modifiers<br>
><br>
> void f(void) {<br>
>      goto end;<br>
>    end:<br>
>      return;<br>
> }<br>
><br>
> _______________________________________________<br>
> cfe-dev mailing list<br>
> <a href="mailto:cfe-dev@cs.uiuc.edu" target="_blank">cfe-dev@cs.uiuc.edu</a><br>
> <a href="http://lists.cs.uiuc.edu/mailman/listinfo/cfe-dev" rel="noreferrer" target="_blank">http://lists.cs.uiuc.edu/mailman/listinfo/cfe-dev</a><br>
<br>
_______________________________________________<br>
cfe-dev mailing list<br>
<a href="mailto:cfe-dev@cs.uiuc.edu" target="_blank">cfe-dev@cs.uiuc.edu</a><br>
<a href="http://lists.cs.uiuc.edu/mailman/listinfo/cfe-dev" rel="noreferrer" target="_blank">http://lists.cs.uiuc.edu/mailman/listinfo/cfe-dev</a><br>
</blockquote></div></div>