From 441460dc9173bc10892ee9f53e5382f5768d4c8f Mon Sep 17 00:00:00 2001 From: Pavel Labath Date: Thu, 5 Apr 2018 15:17:51 +0000 Subject: [PATCH] Remove unused NativeProcessProtocol.h include from Platform.h After removing it, I got a couple of compile errors because we were missing some symbols (SIGKILL and such), as their definitions were not transitively included anymore. I fix this by including csignal from PosixApi.h, as it's windows version provides a stub definitions of these symbols. This should make the result of #including PosixApi.h more consistent across platforms (although in the long run, we should just get rid of this header). llvm-svn: 329296 --- lldb/include/lldb/Host/PosixApi.h | 1 + lldb/include/lldb/Target/Platform.h | 4 ---- 2 files changed, 1 insertion(+), 4 deletions(-) diff --git a/lldb/include/lldb/Host/PosixApi.h b/lldb/include/lldb/Host/PosixApi.h index d5c48dd6d170..ceb41ee040f5 100644 --- a/lldb/include/lldb/Host/PosixApi.h +++ b/lldb/include/lldb/Host/PosixApi.h @@ -18,6 +18,7 @@ #include "lldb/Host/windows/PosixApi.h" #else #include +#include #endif #endif diff --git a/lldb/include/lldb/Target/Platform.h b/lldb/include/lldb/Target/Platform.h index 14a27928c010..fcb9712e89c9 100644 --- a/lldb/include/lldb/Target/Platform.h +++ b/lldb/include/lldb/Target/Platform.h @@ -30,10 +30,6 @@ #include "lldb/lldb-private-forward.h" #include "lldb/lldb-public.h" -// TODO pull NativeDelegate class out of NativeProcessProtocol so we -// can just forward ref the NativeDelegate rather than include it here. -#include "lldb/Host/common/NativeProcessProtocol.h" - namespace lldb_private { class ModuleCache;