jingham added inline comments.

================
Comment at: lldb/source/API/SBData.cpp:12
+#include <memory>
+
 #include "lldb/API/SBData.h"
----------------
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.


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

Reply via email to