From: Mateusz Poliwczak <mpoliwczak34@gmail.com>
Date: Fri, 25 Aug 2023 16:42:42 +0000 (+0000)
Subject: log/slog: use atomic.Pointer instead of atomic.Value
X-Git-Tag: go1.22rc1~1055
X-Git-Url: http://www.git.cypherpunks.su/?a=commitdiff_plain;h=111ab754324e5d5646f84354bc1a134907baaa49;p=gostls13.git

log/slog: use atomic.Pointer instead of atomic.Value

There is no need to use atomic.Value, atomic.Pointer should also
be more performant here.

Change-Id: I04cc1befbe1112665a9d9e16197ea3caef5e5fa3
GitHub-Last-Rev: 577902519ff744c936c5e707802f66aeae32c760
GitHub-Pull-Request: golang/go#62288
Reviewed-on: https://go-review.googlesource.com/c/go/+/523035
TryBot-Result: Gopher Robot <gobot@golang.org>
LUCI-TryBot-Result: Go LUCI <golang-scoped@luci-project-accounts.iam.gserviceaccount.com>
Reviewed-by: Alan Donovan <adonovan@google.com>
Reviewed-by: Jonathan Amsterdam <jba@google.com>
Run-TryBot: Mateusz Poliwczak <mpoliwczak34@gmail.com>
---

diff --git a/src/log/slog/logger.go b/src/log/slog/logger.go
index a068085f47..a42b0a4bcc 100644
--- a/src/log/slog/logger.go
+++ b/src/log/slog/logger.go
@@ -14,14 +14,14 @@ import (
 	"time"
 )
 
-var defaultLogger atomic.Value
+var defaultLogger atomic.Pointer[Logger]
 
 func init() {
 	defaultLogger.Store(New(newDefaultHandler(loginternal.DefaultOutput)))
 }
 
 // Default returns the default Logger.
-func Default() *Logger { return defaultLogger.Load().(*Logger) }
+func Default() *Logger { return defaultLogger.Load() }
 
 // SetDefault makes l the default Logger.
 // After this call, output from the log package's default Logger