[Lldb-commits] [PATCH] D65271: Increase testsuite packet-timeout 5secs -> 1min

Mon Jul 29 09:05:32 PDT 2019

jankratochvil marked 2 inline comments as done.
jankratochvil added inline comments.

================
Comment at: lldb/tools/lldb-test/lldb-test.cpp:979
   ModuleList::GetGlobalModuleListProperties().SetEnableExternalLookup(false);
+  // There is no SB API for settings in general like:
+  // GetGlobalPluginProperties()->SetPacketTimeout(std::chrono::seconds(60));
----------------
labath wrote:
> lldb-test is not confined by the SB API. The reason this (unlike the `symbols.enable-external-lookup` thingy) does not work is because ProcessGDBRemote is a plugin, and so the API to access the setting is hidden by the plugin encapsulation...
Thanks for the explanation; still SB API could provide some function to lookup the plugin by its name to access its settings etc. I have rather deleted the comment there.

Repository:
  rLLDB LLDB

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D65271/new/

https://reviews.llvm.org/D65271