[Lldb-commits] [PATCH] D57990: Use std::make_shared in LLDB (NFC)
Eugene Zelenko via Phabricator via lldb-commits
lldb-commits at lists.llvm.org
Fri Feb 8 18:51:09 PST 2019
Eugene.Zelenko added inline comments.
Comment at: lldb/source/API/SBData.cpp:12
> jingham wrote:
> > Eugene.Zelenko wrote:
> > > Spaces between include statements interfere with Clang-format. Same in other places.
> > How?
> > We always separate system header includes from lldb includes from llvm includes by putting a blank line. That's done all over the place in lldb. I haven't heard of this causing problems before now. If this does cause problems, that should be fixed in clang-format. It seems like a really unreasonable requirement.
> Indeed, and as far as I know llvm does exactly the same thing. clang-format sorts headers within the same group, which is exactly what we want.
Is there any good reason to be different from LLVM/Clang? Wasn't formatting style was changed in not so recent past to reduce differences?
Examples of problems visible in this patch:
CHANGES SINCE LAST ACTION
More information about the lldb-commits