JDevlieghere marked an inline comment as done. JDevlieghere added inline comments.
================ Comment at: lldb/source/API/SBData.cpp:12 +#include <memory> + #include "lldb/API/SBData.h" ---------------- 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. Repository: rLLDB LLDB CHANGES SINCE LAST ACTION https://reviews.llvm.org/D57990/new/ https://reviews.llvm.org/D57990 _______________________________________________ lldb-commits mailing list lldb-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
