oontvoo wrote:

@labath 
> IOW, can we drop the exit_desc field from this struct?

Well, we need a way to distinguish the start and the exit entries 
How about just keeping `exit_code` (which is set to zero)

> I still think it's important to have a simple and succinct way to check 
> whether telemetry is "actually" enabled at a given moment

We can have a `static TelemetryManager::TelemetryCurrentEnabled()` which does 
the check.



https://github.com/llvm/llvm-project/pull/127696
_______________________________________________
lldb-commits mailing list
lldb-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits

Reply via email to