[Lldb-commits] [PATCH] Initialize ProcessGDBRemoteLog for LLGS to fix remote platform logging

Robert Flack flackr at gmail.com
Wed Mar 11 08:09:00 PDT 2015


REPOSITORY
  rL LLVM

================
Comment at: source/Plugins/Process/Linux/ProcessLinux.cpp:64
@@ -71,2 +63,3 @@
+        ProcessPOSIXLog::Initialize();
         ProcessPOSIXLog::RegisterPluginName(GetPluginNameStatic());
     }
----------------
tberghammer wrote:
> Please move this line into ProcessPOSIXLog::Initialize()
Done.

================
Comment at: source/Plugins/Process/POSIX/ProcessPOSIXLog.cpp:41
@@ +40,3 @@
+{
+    static ConstString g_name("linux");
+    static std::once_flag g_once_flag;
----------------
tberghammer wrote:
> Please make it an argument, instead of a hard coded string.
Done.

http://reviews.llvm.org/D8186

EMAIL PREFERENCES
  http://reviews.llvm.org/settings/panel/emailpreferences/






More information about the lldb-commits mailing list