[Lldb-commits] [lldb] [lldb-dap] Create telemetry data only when built with telemetry. (PR #225785)

[Sergei Druzhkov via lldb-commits]

================
@@ -139,26 +141,22 @@ std::string GetSBFileSpecPath(const lldb::SBFileSpec &file_spec);
 lldb::SBLineEntry GetLineEntryForAddress(lldb::SBTarget &target,
                                          const lldb::SBAddress &address);
 
-/// Helper for sending telemetry to lldb server, if client-telemetry is enabled.
-class TelemetryDispatcher {
+namespace detail {
+
+template <bool Enabled = llvm::telemetry::Config::BuildTimeEnableTelemetry>
----------------
DrSergei wrote:

Can we reuse [LLDBConfig](https://github.com/llvm/llvm-project/blob/main/lldb/include/lldb/Core/Telemetry.h#L36) here? I also see runtime setting `enable_client_telemetry` in it, which references `lldb-dap`. Do they conflict with each other?

https://github.com/llvm/llvm-project/pull/225785