]> Cypherpunks.ru repositories - gostls13.git/commitdiff
log/slog: change XXXCtx functions to XXXContext
authorJonathan Amsterdam <jba@google.com>
Thu, 6 Jul 2023 17:06:44 +0000 (13:06 -0400)
committerJonathan Amsterdam <jba@google.com>
Tue, 11 Jul 2023 16:33:46 +0000 (16:33 +0000)
Fixes #61200.

Change-Id: I3071dbf673bcd2d24e62644b40d481c953703978
Reviewed-on: https://go-review.googlesource.com/c/go/+/508195
Run-TryBot: Jonathan Amsterdam <jba@google.com>
Reviewed-by: Russ Cox <rsc@golang.org>
TryBot-Result: Gopher Robot <gobot@golang.org>

api/go1.21.txt
src/log/slog/doc.go
src/log/slog/logger.go
src/log/slog/logger_test.go

index c8ca3df2e65945a54810ce57eb0d5c4a6838b1b9..def0fa84e40482d22ccdd0796dc4a49fe7a5a0f7 100644 (file)
@@ -219,18 +219,18 @@ pkg log/slog, func Any(string, interface{}) Attr #56345
 pkg log/slog, func AnyValue(interface{}) Value #56345
 pkg log/slog, func Bool(string, bool) Attr #56345
 pkg log/slog, func BoolValue(bool) Value #56345
-pkg log/slog, func DebugCtx(context.Context, string, ...interface{}) #56345
+pkg log/slog, func DebugContext(context.Context, string, ...interface{}) #61200
 pkg log/slog, func Debug(string, ...interface{}) #56345
 pkg log/slog, func Default() *Logger #56345
 pkg log/slog, func Duration(string, time.Duration) Attr #56345
 pkg log/slog, func DurationValue(time.Duration) Value #56345
-pkg log/slog, func ErrorCtx(context.Context, string, ...interface{}) #56345
+pkg log/slog, func ErrorContext(context.Context, string, ...interface{}) #61200
 pkg log/slog, func Error(string, ...interface{}) #56345
 pkg log/slog, func Float64(string, float64) Attr #56345
 pkg log/slog, func Float64Value(float64) Value #56345
 pkg log/slog, func Group(string, ...interface{}) Attr #59204
 pkg log/slog, func GroupValue(...Attr) Value #56345
-pkg log/slog, func InfoCtx(context.Context, string, ...interface{}) #56345
+pkg log/slog, func InfoContext(context.Context, string, ...interface{}) #61200
 pkg log/slog, func Info(string, ...interface{}) #56345
 pkg log/slog, func Int64(string, int64) Attr #56345
 pkg log/slog, func Int64Value(int64) Value #56345
@@ -250,7 +250,7 @@ pkg log/slog, func Time(string, time.Time) Attr #56345
 pkg log/slog, func TimeValue(time.Time) Value #56345
 pkg log/slog, func Uint64(string, uint64) Attr #56345
 pkg log/slog, func Uint64Value(uint64) Value #56345
-pkg log/slog, func WarnCtx(context.Context, string, ...interface{}) #56345
+pkg log/slog, func WarnContext(context.Context, string, ...interface{})  #61200
 pkg log/slog, func Warn(string, ...interface{}) #56345
 pkg log/slog, func With(...interface{}) *Logger #56345
 pkg log/slog, method (Attr) Equal(Attr) bool #56345
@@ -271,17 +271,17 @@ pkg log/slog, method (*LevelVar) MarshalText() ([]uint8, error) #56345
 pkg log/slog, method (*LevelVar) Set(Level) #56345
 pkg log/slog, method (*LevelVar) String() string #56345
 pkg log/slog, method (*LevelVar) UnmarshalText([]uint8) error #56345
-pkg log/slog, method (*Logger) DebugCtx(context.Context, string, ...interface{}) #56345
+pkg log/slog, method (*Logger) DebugContext(context.Context, string, ...interface{}) #61200
 pkg log/slog, method (*Logger) Debug(string, ...interface{}) #56345
 pkg log/slog, method (*Logger) Enabled(context.Context, Level) bool #56345
-pkg log/slog, method (*Logger) ErrorCtx(context.Context, string, ...interface{}) #56345
+pkg log/slog, method (*Logger) ErrorContext(context.Context, string, ...interface{}) #61200
 pkg log/slog, method (*Logger) Error(string, ...interface{}) #56345
 pkg log/slog, method (*Logger) Handler() Handler #56345
-pkg log/slog, method (*Logger) InfoCtx(context.Context, string, ...interface{}) #56345
+pkg log/slog, method (*Logger) InfoContext(context.Context, string, ...interface{}) #61200
 pkg log/slog, method (*Logger) Info(string, ...interface{}) #56345
 pkg log/slog, method (*Logger) LogAttrs(context.Context, Level, string, ...Attr) #56345
 pkg log/slog, method (*Logger) Log(context.Context, Level, string, ...interface{}) #56345
-pkg log/slog, method (*Logger) WarnCtx(context.Context, string, ...interface{}) #56345
+pkg log/slog, method (*Logger) WarnContext(context.Context, string, ...interface{}) #61200
 pkg log/slog, method (*Logger) Warn(string, ...interface{}) #56345
 pkg log/slog, method (*Logger) WithGroup(string) *Logger #56345
 pkg log/slog, method (*Logger) With(...interface{}) *Logger #56345
index 97c1a0b5ef8e0e41bfb5e7adb140a19dcb6aadca..d7a2bacb896e013e1da8cdec152198d0ca0bc991 100644 (file)
@@ -174,9 +174,9 @@ argument, as do their corresponding top-level functions.
 
 Although the convenience methods on Logger (Info and so on) and the
 corresponding top-level functions do not take a context, the alternatives ending
-in "Ctx" do. For example,
+in "Context" do. For example,
 
-       slog.InfoCtx(ctx, "message")
+       slog.InfoContext(ctx, "message")
 
 It is recommended to pass a context to an output method if one is available.
 
index 2bad5dfccc91adef95468fdb7b4f070212bf7873..b6fea60c9c03897ab46e68221aab841667c6cb1a 100644 (file)
@@ -168,8 +168,8 @@ func (l *Logger) Debug(msg string, args ...any) {
        l.log(nil, LevelDebug, msg, args...)
 }
 
-// DebugCtx logs at LevelDebug with the given context.
-func (l *Logger) DebugCtx(ctx context.Context, msg string, args ...any) {
+// DebugContext logs at LevelDebug with the given context.
+func (l *Logger) DebugContext(ctx context.Context, msg string, args ...any) {
        l.log(ctx, LevelDebug, msg, args...)
 }
 
@@ -178,8 +178,8 @@ func (l *Logger) Info(msg string, args ...any) {
        l.log(nil, LevelInfo, msg, args...)
 }
 
-// InfoCtx logs at LevelInfo with the given context.
-func (l *Logger) InfoCtx(ctx context.Context, msg string, args ...any) {
+// InfoContext logs at LevelInfo with the given context.
+func (l *Logger) InfoContext(ctx context.Context, msg string, args ...any) {
        l.log(ctx, LevelInfo, msg, args...)
 }
 
@@ -188,8 +188,8 @@ func (l *Logger) Warn(msg string, args ...any) {
        l.log(nil, LevelWarn, msg, args...)
 }
 
-// WarnCtx logs at LevelWarn with the given context.
-func (l *Logger) WarnCtx(ctx context.Context, msg string, args ...any) {
+// WarnContext logs at LevelWarn with the given context.
+func (l *Logger) WarnContext(ctx context.Context, msg string, args ...any) {
        l.log(ctx, LevelWarn, msg, args...)
 }
 
@@ -198,8 +198,8 @@ func (l *Logger) Error(msg string, args ...any) {
        l.log(nil, LevelError, msg, args...)
 }
 
-// ErrorCtx logs at LevelError with the given context.
-func (l *Logger) ErrorCtx(ctx context.Context, msg string, args ...any) {
+// ErrorContext logs at LevelError with the given context.
+func (l *Logger) ErrorContext(ctx context.Context, msg string, args ...any) {
        l.log(ctx, LevelError, msg, args...)
 }
 
@@ -250,8 +250,8 @@ func Debug(msg string, args ...any) {
        Default().log(nil, LevelDebug, msg, args...)
 }
 
-// DebugCtx calls Logger.DebugCtx on the default logger.
-func DebugCtx(ctx context.Context, msg string, args ...any) {
+// DebugContext calls Logger.DebugContext on the default logger.
+func DebugContext(ctx context.Context, msg string, args ...any) {
        Default().log(ctx, LevelDebug, msg, args...)
 }
 
@@ -260,8 +260,8 @@ func Info(msg string, args ...any) {
        Default().log(nil, LevelInfo, msg, args...)
 }
 
-// InfoCtx calls Logger.InfoCtx on the default logger.
-func InfoCtx(ctx context.Context, msg string, args ...any) {
+// InfoContext calls Logger.InfoContext on the default logger.
+func InfoContext(ctx context.Context, msg string, args ...any) {
        Default().log(ctx, LevelInfo, msg, args...)
 }
 
@@ -270,8 +270,8 @@ func Warn(msg string, args ...any) {
        Default().log(nil, LevelWarn, msg, args...)
 }
 
-// WarnCtx calls Logger.WarnCtx on the default logger.
-func WarnCtx(ctx context.Context, msg string, args ...any) {
+// WarnContext calls Logger.WarnContext on the default logger.
+func WarnContext(ctx context.Context, msg string, args ...any) {
        Default().log(ctx, LevelWarn, msg, args...)
 }
 
@@ -280,8 +280,8 @@ func Error(msg string, args ...any) {
        Default().log(nil, LevelError, msg, args...)
 }
 
-// ErrorCtx calls Logger.ErrorCtx on the default logger.
-func ErrorCtx(ctx context.Context, msg string, args ...any) {
+// ErrorContext calls Logger.ErrorContext on the default logger.
+func ErrorContext(ctx context.Context, msg string, args ...any) {
        Default().log(ctx, LevelError, msg, args...)
 }
 
index 130f2e6ac8a5300f09486afa0bc43e457f0c1885..b8a01dcb4de8665b7c52172e940472a799fc0d10 100644 (file)
@@ -395,14 +395,14 @@ func TestContext(t *testing.T) {
                f         func(context.Context, string, ...any)
                wantLevel Level
        }{
-               {l.DebugCtx, LevelDebug},
-               {l.InfoCtx, LevelInfo},
-               {l.WarnCtx, LevelWarn},
-               {l.ErrorCtx, LevelError},
-               {DebugCtx, LevelDebug},
-               {InfoCtx, LevelInfo},
-               {WarnCtx, LevelWarn},
-               {ErrorCtx, LevelError},
+               {l.DebugContext, LevelDebug},
+               {l.InfoContext, LevelInfo},
+               {l.WarnContext, LevelWarn},
+               {l.ErrorContext, LevelError},
+               {DebugContext, LevelDebug},
+               {InfoContext, LevelInfo},
+               {WarnContext, LevelWarn},
+               {ErrorContext, LevelError},
        } {
                h.clear()
                ctx := context.WithValue(context.Background(), "L", test.wantLevel)