[lldb] ProcessGdbRemote header gardning

Remove unused headers, add used headers, remove declared-but-not-defined
entities.

Author: labath

lldb/source/Plugins/Process/gdb-remote/GDBRemoteClientBase.h

@@ -10,8 +10,13 @@
 #define LLDB_SOURCE_PLUGINS_PROCESS_GDB_REMOTE_GDBREMOTECLIENTBASE_H
 
 #include "GDBRemoteCommunication.h"
-
+#include "lldb/Utility/Broadcaster.h"
+#include "llvm/ADT/STLFunctionalExtras.h"
+#include "llvm/ADT/StringRef.h"
+#include <chrono>
 #include <condition_variable>
+#include <cstdint>
+#include <mutex>
 
 namespace lldb_private {
 namespace process_gdb_remote {

lldb/source/Plugins/Process/gdb-remote/GDBRemoteCommunication.cpp

@@ -7,14 +7,7 @@
 //===----------------------------------------------------------------------===//
 
 #include "GDBRemoteCommunication.h"
-
-#include <climits>
-#include <cstring>
-#include <future>
-#include <sys/stat.h>
-
-#include "lldb/Host/Config.h"
-#include "lldb/Host/ConnectionFileDescriptor.h"
+#include "ProcessGDBRemoteLog.h"
 #include "lldb/Host/FileSystem.h"
 #include "lldb/Host/Host.h"
 #include "lldb/Host/HostInfo.h"
@@ -30,13 +23,14 @@
 #include "lldb/Utility/Log.h"
 #include "lldb/Utility/RegularExpression.h"
 #include "lldb/Utility/StreamString.h"
-#include "llvm/ADT/ArrayRef.h"
 #include "llvm/ADT/SmallString.h"
 #include "llvm/ADT/StringRef.h"
 #include "llvm/Config/llvm-config.h" // for LLVM_ENABLE_ZLIB
 #include "llvm/Support/ScopedPrinter.h"
-
-#include "ProcessGDBRemoteLog.h"
+#include <climits>
+#include <cstring>
+#include <future>
+#include <sys/stat.h>
 
 #if defined(__APPLE__)
 #define DEBUGSERVER_BASENAME "debugserver"

lldb/source/Plugins/Process/gdb-remote/GDBRemoteCommunication.h

@@ -11,27 +11,16 @@
 
 #include "GDBRemoteCommunicationHistory.h"
 
-#include <condition_variable>
 #include <future>
 #include <mutex>
-#include <queue>
 #include <string>
-#include <vector>
-
 #include "lldb/Core/Communication.h"
-#include "lldb/Host/Config.h"
 #include "lldb/Host/HostThread.h"
 #include "lldb/Host/Socket.h"
 #include "lldb/Utility/Args.h"
-#include "lldb/Utility/Listener.h"
-#include "lldb/Utility/Predicate.h"
 #include "lldb/Utility/StringExtractorGDBRemote.h"
-#include "lldb/lldb-public.h"
 
 namespace lldb_private {
-namespace repro {
-class PacketRecorder;
-}
 namespace process_gdb_remote {
 
 enum GDBStoppointType {
@@ -162,8 +151,6 @@ class GDBRemoteCommunication : public Communication {
 
   void DumpHistory(Stream &strm);
 
-  void SetPacketRecorder(repro::PacketRecorder *recorder);
-
   static llvm::Error ConnectLocally(GDBRemoteCommunication &client,
                                     GDBRemoteCommunication &server);
 

lldb/unittests/Process/gdb-remote/GDBRemoteClientBaseTest.cpp

@@ -5,16 +5,20 @@
 // SPDX-License-Identifier: Apache-2.0 WITH LLVM-exception
 //
 //===----------------------------------------------------------------------===//
-#include <future>
 
+#include "Plugins/Process/gdb-remote/GDBRemoteClientBase.h"
 #include "GDBRemoteTestUtils.h"
-
 #include "Plugins/Process/Utility/LinuxSignals.h"
-#include "Plugins/Process/gdb-remote/GDBRemoteClientBase.h"
 #include "Plugins/Process/gdb-remote/GDBRemoteCommunicationServer.h"
 #include "lldb/Utility/GDBRemote.h"
-#include "llvm/ADT/STLExtras.h"
+#include "lldb/Utility/Listener.h"
+#include "llvm/ADT/StringRef.h"
 #include "llvm/Testing/Support/Error.h"
+#include "gtest/gtest.h"
+#include <chrono>
+#include <future>
+#include <string>
+#include <vector>
 
 using namespace lldb_private::process_gdb_remote;
 using namespace lldb_private;