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



_______________________________________________
lldb-commits mailing list
lldb-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits

Reply via email to