[Lldb-commits] [PATCH] D35113: Clean up lldb-types.h

[Pavel Labath via Phabricator via lldb-commits]

labath added a comment.





================
Comment at: include/lldb/Host/MainLoop.h:16
 #include "llvm/ADT/DenseMap.h"
+#include <signal.h>
 
----------------
Eugene.Zelenko wrote:
> I think will be good idea to include csignal instead. Same in other files. See http://clang.llvm.org/extra/clang-tidy/checks/modernize-deprecated-headers.html.
Good idea. I wasn't aware <csignal> existed.


https://reviews.llvm.org/D35113