<div dir="ltr"><div dir="ltr">Hey Pavel,<div><br></div><div>Sounds like a good idea. I don't have a strong opinion on this matter, but I'm always in favor of improving readability.</div><div><br></div><div>Cheers,</div><div>Jonas</div></div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Wed, Apr 17, 2019 at 3:38 AM Pavel Labath via lldb-dev <<a href="mailto:lldb-dev@lists.llvm.org">lldb-dev@lists.llvm.org</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-style:solid;border-left-color:rgb(204,204,204);padding-left:1ex">Hello all,<br>
<br>
some llvm classes, are so well-known and widely used, that qualifying <br>
them with "llvm::" serves no useful purpose and only adds visual noise. <br>
I'm thinking here mainly of ADT classes like String/ArrayRef, <br>
Optional/Error, etc. I propose we stop explicitly qualifying these classes.<br>
<br>
We can implement this proposal the same way as clang solved the same <br>
problem, which is by creating a special LLVM.h <br>
<<a href="https://github.com/llvm-mirror/clang/blob/master/include/clang/Basic/LLVM.h" rel="noreferrer" target="_blank">https://github.com/llvm-mirror/clang/blob/master/include/clang/Basic/LLVM.h</a>> <br>
header in the Utility library. This header would adopt these classes <br>
into the lldb_private namespace via a series of forward and "using" <br>
declarations.<br>
<br>
I think clang's LLVM.h is contains a well-balanced collection of adopted <br>
classes, and it should cover the most widely-used classes in lldb too, <br>
so I propose we use that as a starting point.<br>
<br>
What do you think?<br>
<br>
regards,<br>
pavel<br>
<br>
PS: I'm not proposing any wholesale removal of "llvm::" qualifiers from <br>
these types, though I may do some smaller-scale removals if I'm about to <br>
substantially modify a file.<br>
_______________________________________________<br>
lldb-dev mailing list<br>
<a href="mailto:lldb-dev@lists.llvm.org" target="_blank">lldb-dev@lists.llvm.org</a><br>
<a href="https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-dev" rel="noreferrer" target="_blank">https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-dev</a><br>
</blockquote></div>