[Lldb-commits] [PATCH] D11094: Refactor Unix signals.

Chaoren Lin chaorenl at google.com
Mon Jul 13 10:28:17 PDT 2015


chaoren added inline comments.

================
Comment at: source/Plugins/Platform/gdb-server/PlatformRemoteGDBServer.cpp:978
@@ +977,3 @@
+
+    static const auto s_default_unix_signals_sp = std::make_shared<UnixSignals>();
+    return s_default_unix_signals_sp;
----------------
ovyalov wrote:
> Should it be Host::GetUnixSignals?
I think it's better to use the default Unix signal set without any assumptions about the remote platform, than to assume that the remote platform would be the same as the host.


http://reviews.llvm.org/D11094







More information about the lldb-commits mailing list