<div dir="ltr"><br><div class="gmail_extra"><br><br><div class="gmail_quote">On Sun, Oct 13, 2013 at 11:43 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"><div class="im">On Fri, Oct 11, 2013 at 9:02 PM, Daniel Jasper <<a href="mailto:djasper@google.com" class="cremed">djasper@google.com</a>> wrote:<br>

><br>
>> 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>
><br>
><br>
> I would guess that this change is much lower value. Usually you would use<br>
> clang-format mostly within editors or on patches that you create and these<br>
> will rarely contain the namespace declarations. If they do, it is a matter<br>
> of hitting "J" a few times in vim ;-).<br>
<br>
</div>We're considering running clang-format in batch over our entire<br>
codebase, so we're coming at this from a slightly different angle.<br></blockquote><div><br></div><div>Well, that would also be a good opportunity to get rid of this particular coding style ;-)..</div><div><br></div>
<div>Slightly more serious, I am all for preserving vertical space as I think it does matter how much code fits onto a display page. However, such namespace declarations are almost always in irrelevant parts of the file and the gain of putting them all in one line is really small.</div>
<div><br></div><div>Cheers,<br>Daniel</div><div><br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
I'll see if I can come up with a concrete patch at some point, and<br>
we'll see what the impact is like.<br>
<span class="HOEnZb"><font color="#888888"><br>
- Kim<br>
</font></span></blockquote></div><br></div></div>