]> Cypherpunks repositories - gostls13.git/commitdiff
internal/trace: fix Sync.ClockSnapshot comment
authorFelix Geisendörfer <felix.geisendoerfer@datadoghq.com>
Mon, 11 Aug 2025 06:33:05 +0000 (08:33 +0200)
committerGopher Robot <gobot@golang.org>
Tue, 12 Aug 2025 18:42:10 +0000 (11:42 -0700)
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 <mknyszek@google.com>
LUCI-TryBot-Result: Go LUCI <golang-scoped@luci-project-accounts.iam.gserviceaccount.com>
Reviewed-by: David Chase <drchase@google.com>
Auto-Submit: Michael Knyszek <mknyszek@google.com>

src/internal/trace/event.go

index f31412e35d889afef2c1292b7ca59658d36781c6..f9c40010c4b535620caaff7ebae17b8705092795 100644 (file)
@@ -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.