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

[Eugene Zelenko via Phabricator via lldb-commits]

Eugene.Zelenko added inline comments.


================
Comment at: include/lldb/Host/MainLoop.h:16
 #include "llvm/ADT/DenseMap.h"
+#include <signal.h>
 
----------------
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.


https://reviews.llvm.org/D35113