1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 20 21 22 23 24 25 26
|
From: Mathias Gibbens <gibmat@debian.org>
Description: Place fields used in atomic operations at beginning of struct
Forwarded: https://github.com/olekukonko/errors/pull/3
diff --git a/errors.go b/errors.go
index 4f6509d..5158ae3 100644
--- a/errors.go
+++ b/errors.go
@@ -95,6 +95,10 @@ type contextItem struct {
// context, cause, and metadata like code and category. It is thread-safe and
// supports pooling for performance.
type Error struct {
+ // Fields used in atomic operations. Place them at the beginning of the
+ // struct to ensure proper alignment across all architectures.
+ count uint64 // Occurrence count for tracking frequency.
+
// Primary fields (frequently accessed).
msg string // The error message displayed by Error().
name string // The error name or type (e.g., "AuthError").
@@ -103,7 +107,6 @@ type Error struct {
// Secondary metadata.
template string // Fallback message template if msg is empty.
category string // Error category (e.g., "network").
- count uint64 // Occurrence count for tracking frequency.
code int32 // HTTP-like status code (e.g., 400, 500).
smallCount int32 // Number of items in smallContext.
|