<div dir="ltr"><br><div class="gmail_extra"><br><br><div class="gmail_quote">On Fri, Oct 11, 2013 at 1:35 AM, Kim Gräsman <span dir="ltr"><<a href="mailto:kim.grasman@gmail.com" target="_blank" class="cremed">kim.grasman@gmail.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Thanks, all!<br>
<div class="im"><br>
On Thu, Oct 10, 2013 at 5:26 PM, Sean Silva <<a href="mailto:silvas@purdue.edu" class="cremed">silvas@purdue.edu</a>> wrote:<br>
>>><br>
>>> 1) As far as I can tell the indentation for continuation lines is<br>
>>> always 4. Is there a strong reason, or could we add a style option for<br>
>>> it?<br>
>><br>
>> I don't see why it should be a fixed option. Looks like no one bothered<br>
>> changing. ;)<br>
><br>
><br>
> I've recently encountered one project where the coding standard requires<br>
> that continuation lines be indented by 1 space, and I found this to be an<br>
> issue with clang-format's formatting as well; it would be great to make this<br>
> configurable :)<br>
<br>
</div>I have a sketchy patch for this that I'll try and clean up and post<br>
for review as soon as possible.<br></blockquote><div><br></div><div>Cool. I certainly agree that this should be an option. We just never got around to making it one.</div><div><br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">

The namespace declaration style may be more work, so I'll see if I can<br>
find some time to figure it out once I'm better acquainted with the<br>
code base.<br></blockquote><div><br></div><div>I would guess that this change is much lower value. Usually you would use clang-format mostly within editors or on patches that you create and these will rarely contain the namespace declarations. If they do, it is a matter of hitting "J" a few times in vim ;-). </div>
<div><br></div><div>Cheers,<br>Daniel</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
Cheers,<br>
- Kim<br>
<div class="HOEnZb"><div class="h5">_______________________________________________<br>
cfe-dev mailing list<br>
<a href="mailto:cfe-dev@cs.uiuc.edu" class="cremed">cfe-dev@cs.uiuc.edu</a><br>
<a href="http://lists.cs.uiuc.edu/mailman/listinfo/cfe-dev" target="_blank" class="cremed">http://lists.cs.uiuc.edu/mailman/listinfo/cfe-dev</a><br>
</div></div></blockquote></div><br></div></div>