[Lldb-commits] [PATCH] D13574: [LLDB] Fix Clang-tidy misc-use-override warnings in some files in include/lldb/Core; other minor fixes.

Adrian McCarthy via lldb-commits lldb-commits at lists.llvm.org
Fri Oct 9 11:09:32 PDT 2015


There was an lldb-dev thread from February 9, where it seemed to be agreed
that lldb code should be more like llvm code, in particular with the spaces
between the function names and the opening parens.  I've been omitting the
spaces in new code since then.

On Fri, Oct 9, 2015 at 10:14 AM, Greg Clayton via lldb-commits <
lldb-commits at lists.llvm.org> wrote:

> clayborg accepted this revision.
> clayborg added a comment.
> This revision is now accepted and ready to land.
>
> We are pretty inconsistent with the space after a function name and before
> the ( so I don't mind.
>
>
> Repository:
>   rL LLVM
>
> http://reviews.llvm.org/D13574
>
>
>
> _______________________________________________
> lldb-commits mailing list
> lldb-commits at lists.llvm.org
> http://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/lldb-commits/attachments/20151009/0098f0cb/attachment.html>


More information about the lldb-commits mailing list