From f63e12d0e0f4533af54550d6d049a836fb738147 Mon Sep 17 00:00:00 2001 From: =?utf8?q?Felix=20Geisend=C3=B6rfer?= Date: Mon, 11 Aug 2025 08:33:05 +0200 Subject: [PATCH] internal/trace: fix Sync.ClockSnapshot comment The old comment said "clocks take in close in time" which was probably due to rewording this a few times. Replace the comment with the one of the ClockSnapshot type as there doesn't seem to be a good reason for using a different wording here. Change-Id: I6a6a69648c8470c2f45f6f8e728f5dc8b121a82b Reviewed-on: https://go-review.googlesource.com/c/go/+/694620 Reviewed-by: Michael Knyszek LUCI-TryBot-Result: Go LUCI Reviewed-by: David Chase Auto-Submit: Michael Knyszek --- src/internal/trace/event.go | 8 +++++--- 1 file changed, 5 insertions(+), 3 deletions(-) diff --git a/src/internal/trace/event.go b/src/internal/trace/event.go index f31412e35d..f9c40010c4 100644 --- a/src/internal/trace/event.go +++ b/src/internal/trace/event.go @@ -690,9 +690,11 @@ type Sync struct { // N indicates that this is the Nth sync event in the trace. N int - // ClockSnapshot is a snapshot of different clocks taken in close in time - // that can be used to correlate trace events with data captured by other - // tools. May be nil for older trace versions. + // ClockSnapshot represents a near-simultaneous clock reading of several + // different system clocks. The snapshot can be used as a reference to + // convert timestamps to different clocks, which is helpful for correlating + // timestamps with data captured by other tools. The value is nil for traces + // before go1.25. ClockSnapshot *ClockSnapshot // ExperimentalBatches contain all the unparsed batches of data for a given experiment. -- 2.51.0