[go] log/slog: Use B.Loop to simplify the code

0 views
Skip to first unread message

David Chase (Gerrit)

unread,
Feb 24, 2026, 3:48:39 PM (12 hours ago) Feb 24
to Gerrit Bot, goph...@pubsubhelper.golang.org, golang-...@googlegroups.com, Jonathan Amsterdam, Go LUCI, qiu laidongfeng, kyose chang, Gopher Robot, golang-co...@googlegroups.com

David Chase submitted the change

Change information

Commit message:
log/slog: use B.Loop to simplify the code

Simplify the code and remove some unnecessary calls of ResetTimer.

Updates #61515
Change-Id: Ifbcd8deb1aa818b3ee190dab537250ba15b4cdc0
GitHub-Last-Rev: 397c230a35d928e9c22ab8ead354e2c73ea8b98c
GitHub-Pull-Request: golang/go#77592
Reviewed-by: Jonathan Amsterdam <j...@google.com>
Reviewed-by: David Chase <drc...@google.com>
Files:
  • M src/log/slog/handler_test.go
  • M src/log/slog/json_handler_test.go
  • M src/log/slog/level_test.go
  • M src/log/slog/value_access_benchmark_test.go
  • M src/log/slog/value_test.go
Change size: S
Delta: 5 files changed, 5 insertions(+), 11 deletions(-)
Branch: refs/heads/master
Submit Requirements:
  • requirement satisfiedCode-Review: +2 by Jonathan Amsterdam, +1 by David Chase
  • requirement satisfiedTryBots-Pass: LUCI-TryBot-Result+1 by Go LUCI
Open in Gerrit
Inspect html for hidden footers to help with email filtering. To unsubscribe visit settings. DiffyGerrit
Gerrit-MessageType: merged
Gerrit-Project: go
Gerrit-Branch: master
Gerrit-Change-Id: Ifbcd8deb1aa818b3ee190dab537250ba15b4cdc0
Gerrit-Change-Number: 744881
Gerrit-PatchSet: 3
Gerrit-Owner: Gerrit Bot <letsus...@gmail.com>
Gerrit-Reviewer: David Chase <drc...@google.com>
Gerrit-Reviewer: Jonathan Amsterdam <j...@google.com>
Gerrit-Reviewer: qiu laidongfeng <26454...@qq.com>
Gerrit-CC: Gopher Robot <go...@golang.org>
Gerrit-CC: kyose chang <sev...@gmail.com>
open
diffy
satisfied_requirement
Reply all
Reply to author
Forward
0 new messages