[lldb][NFC] Remove an outdated comment in HostInfoBase
authorRaphael Isemann <teemperor@gmail.com>
Tue, 22 Jun 2021 14:45:48 +0000 (16:45 +0200)
committerRaphael Isemann <teemperor@gmail.com>
Tue, 22 Jun 2021 14:48:17 +0000 (16:48 +0200)
We should *never* use static local variables in this file as this makes
unittesting the plugin code impossible (and this whole 'testing' thing has
turned out to be rather useful so far).

lldb/source/Host/common/HostInfoBase.cpp

index d923a40..a6239a3 100644 (file)
@@ -31,12 +31,7 @@ using namespace lldb;
 using namespace lldb_private;
 
 namespace {
-// The HostInfoBaseFields is a work around for windows not supporting static
-// variables correctly in a thread safe way. Really each of the variables in
-// HostInfoBaseFields should live in the functions in which they are used and
-// each one should be static, but the work around is in place to avoid this
-// restriction. Ick.
-
+/// Contains the state of the HostInfoBase plugin.
 struct HostInfoBaseFields {
   ~HostInfoBaseFields() {
     if (FileSystem::Instance().Exists(m_lldb_process_tmp_dir)) {