<div dir="ltr">Agreed. Before I clang-formatted it, it looked fine, because each definition was on a single line. But clang-format made them 3 lines each. So yea, we should definitely merge them. I was copy/pasting the output from IWYU, which is why it looks this way. Thanks for bringing it up</div><br><div class="gmail_quote"><div dir="ltr">On Thu, Apr 6, 2017 at 3:32 PM Tamas Berghammer via Phabricator <<a href="mailto:reviews@reviews.llvm.org">reviews@reviews.llvm.org</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">tberghammer added subscribers: lldb-commits, tberghammer.<br>
tberghammer added inline comments.<br>
<br>
/lldb/trunk/include/lldb/Core/Address.h:21-50 I think we should try to merge these namespace definitions as in my view the current syntax makes the top of the file very noisy. What do you think?<br>
<br>
Users:<br>
zturner (Author)<br>
<br>
<a href="https://reviews.llvm.org/rL299714" rel="noreferrer" target="_blank">https://reviews.llvm.org/rL299714</a><br>
<br>
<br>
<br>
</blockquote></div>