format

Author: oontvoo

lldb/include/lldb/Core/Telemetry.h

@@ -22,8 +22,8 @@
 #include <ctime>
 #include <memory>
 #include <optional>
-#include <string>
 #include <stack>
+#include <string>
 
 namespace lldb_private {
 namespace telemetry {
@@ -91,7 +91,6 @@ class TelemetryManager : public llvm::telemetry::Manager {
 public:
   llvm::Error preDispatch(llvm::telemetry::TelemetryInfo *entry) override;
 
-
   virtual void AtDebuggerStartup(DebuggerInfo *entry);
   virtual void AtDebuggerExit(DebuggerInfo *entry);
 
@@ -115,38 +114,36 @@ class TelemetryManager : public llvm::telemetry::Manager {
 
 /// Helper RAII class for collecting telemetry.
 class ScopeTelemetryCollector {
- public:
-  ScopeTelemetryCollector ()  {
+public:
+  ScopeTelemetryCollector() {
     if (TelemetryEnabled())
       m_start = std::chrono::steady_clock::now();
   }
 
   ~ScopeTelemetryCollector() {
-    while(! m_exit_funcs.empty()) {
+    while (!m_exit_funcs.empty()) {
       (m_exit_funcs.top())();
       m_exit_funcs.pop();
     }
   }
 
   bool TelemetryEnabled() {
-    TelemetryManager* instance = TelemetryManager::GetInstance();
-  return instance != nullptr && instance->GetConfig()->EnableTelemetry;
+    TelemetryManager *instance = TelemetryManager::GetInstance();
+    return instance != nullptr && instance->GetConfig()->EnableTelemetry;
   }
 
+  SteadyTimePoint GetStartTime() { return m_start; }
+  SteadyTimePoint GetCurrentTime() { return std::chrono::steady_clock::now(); }
 
-  SteadyTimePoint GetStartTime() {return m_start;}
-  SteadyTimePoint GetCurrentTime()  { return std::chrono::steady_clock::now(); }
-
- template <typename Fp>
- void RunAtScopeExit(Fp&& F){
-   assert(llvm::telemetry::Config::BuildTimeEnableTelemetry && "Telemetry should have been enabled");
-   m_exit_funcs.push(std::forward<Fp>(F));
- }
+  template <typename Fp> void RunAtScopeExit(Fp &&F) {
+    assert(llvm::telemetry::Config::BuildTimeEnableTelemetry &&
+           "Telemetry should have been enabled");
+    m_exit_funcs.push(std::forward<Fp>(F));
+  }
 
- private:
+private:
   SteadyTimePoint m_start;
   std::stack<std::function<void()>> m_exit_funcs;
-
 };
 
 } // namespace telemetry

lldb/source/Core/Debugger.cpp

@@ -17,6 +17,7 @@
 #include "lldb/Core/PluginManager.h"
 #include "lldb/Core/Progress.h"
 #include "lldb/Core/StreamAsynchronousIO.h"
+#include "lldb/Core/Telemetry.h"
 #include "lldb/DataFormatters/DataVisualization.h"
 #include "lldb/Expression/REPL.h"
 #include "lldb/Host/File.h"
@@ -52,7 +53,6 @@
 #include "lldb/Utility/State.h"
 #include "lldb/Utility/Stream.h"
 #include "lldb/Utility/StreamString.h"
-#include "lldb/Core/Telemetry.h"
 #include "lldb/lldb-enumerations.h"
 
 #if defined(_WIN32)
@@ -71,7 +71,6 @@
 #include "llvm/Support/Threading.h"
 #include "llvm/Support/raw_ostream.h"
 
-
 #include <chrono>
 #include <cstdio>
 #include <cstdlib>
@@ -769,15 +768,15 @@ DebuggerSP Debugger::CreateInstance(lldb::LogOutputCallback log_callback,
   DebuggerSP debugger_sp(new Debugger(log_callback, baton));
 
   if (helper.TelemetryEnabled()) {
-    helper.RunAtScopeExit([&](){
-      lldb_private::telemetry::TelemetryManager * manager = lldb_private::telemetry::TelemetryManager::GetInstance();
+    helper.RunAtScopeExit([&]() {
+      lldb_private::telemetry::TelemetryManager *manager =
+          lldb_private::telemetry::TelemetryManager::GetInstance();
       lldb_private::telemetry::DebuggerInfo entry;
       entry.debugger = debugger_sp.get();
       entry.start_time = helper.GetStartTime();
       entry.end_time = helper.GetCurrentTime();
       manager->AtDebuggerStartup(&entry);
     });
-
   }
 
   if (g_debugger_list_ptr && g_debugger_list_mutex_ptr) {
@@ -1009,9 +1008,9 @@ void Debugger::Clear() {
     lldb_private::telemetry::ScopeTelemetryCollector helper;
     if (helper.TelemetryEnabled()) {
       // TBD: We *may* have to send off the log BEFORE the ClearIOHanders()?
-      helper.RunAtScopeExit([&helper, this](){
-        lldb_private::telemetry::TelemetryManager* manager
-            = lldb_private::telemetry::TelemetryManager::GetInstance();
+      helper.RunAtScopeExit([&helper, this]() {
+        lldb_private::telemetry::TelemetryManager *manager =
+            lldb_private::telemetry::TelemetryManager::GetInstance();
         lldb_private::telemetry::DebuggerInfo entry;
         entry.debugger = this;
         entry.exit_desc = {0, ""}; // If we are here, there was no error.

lldb/source/Core/Telemetry.cpp

@@ -96,7 +96,7 @@ void TelemetryManager::AtDebuggerStartup(DebuggerInfo *entry) {
   }
 }
 
-void TelemetryManager::AtDebuggerExit(DebuggerInfo * entry) {
+void TelemetryManager::AtDebuggerExit(DebuggerInfo *entry) {
   // There must be a reference to the debugger at this point.
   assert(entry->debugger != nullptr);
 
@@ -119,6 +119,5 @@ void TelemetryManager::SetInstance(std::unique_ptr<TelemetryManager> manager) {
   g_instance = std::move(manager);
 }
 
-
 } // namespace telemetry
 } // namespace lldb_private