Reorder members to avoid unaligned atomic operations on 32-bit archs

influxdata · Dec 6, 2023 · 4f71e65 · 4f71e65
1 parent bfcdfdd
commit 4f71e65
Showing 1 changed file with 4 additions and 4 deletions.
diff --git a/testutil/accumulator.go b/testutil/accumulator.go
@@ -30,19 +30,19 @@ func (p *Metric) String() string {
 
 // Accumulator defines a mocked out accumulator
 type Accumulator struct {
-	sync.Mutex
-	*sync.Cond
-
+	nMetrics    uint64 // Needs to be first to avoid unaligned atomic operations on 32-bit archs
 	Metrics     []*Metric
 	accumulated []telegraf.Metric
-	nMetrics    uint64
 	Discard     bool
 	Errors      []error
 	debug       bool
 	deliverChan chan telegraf.DeliveryInfo
 	delivered   []telegraf.DeliveryInfo
 
 	TimeFunc func() time.Time
+
+	sync.Mutex
+	*sync.Cond
 }
 
 func (a *Accumulator) NMetrics() uint64 {