[go] hash: use more internal/byteorder functions to simplify the code

3 views
Skip to first unread message

Gopher Robot (Gerrit)

unread,
May 18, 2024, 6:36:48 PMMay 18
to Gerrit Bot, goph...@pubsubhelper.golang.org, golang-...@googlegroups.com, Go LUCI, Dmitri Shuralyov, Dmitri Shuralyov, Keith Randall, kyose chang, golang-co...@googlegroups.com

Gopher Robot submitted the change

Change information

Commit message:
hash: use more internal/byteorder functions to simplify the code

A follow-up for the CL 585015.
Change-Id: I412f33f1d75abe1446cb3fd742d44d3cb4350380
GitHub-Last-Rev: 554ace757cc75389e7a3c441d3a60cfa85c815a5
GitHub-Pull-Request: golang/go#67476
Reviewed-by: Dmitri Shuralyov <dmit...@google.com>
Reviewed-by: Ian Lance Taylor <ia...@google.com>
Reviewed-by: Keith Randall <k...@golang.org>
Auto-Submit: Ian Lance Taylor <ia...@google.com>
Auto-Submit: Dmitri Shuralyov <dmit...@golang.org>
Commit-Queue: Ian Lance Taylor <ia...@google.com>
Files:
  • M src/hash/crc32/crc32_generic.go
  • M src/hash/crc64/crc64.go
  • M src/hash/fnv/fnv.go
Change size: S
Delta: 3 files changed, 12 insertions(+), 15 deletions(-)
Branch: refs/heads/master
Submit Requirements:
  • requirement satisfiedCode-Review: +1 by Dmitri Shuralyov, +2 by Ian Lance Taylor, +2 by Keith Randall
  • 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: I412f33f1d75abe1446cb3fd742d44d3cb4350380
Gerrit-Change-Number: 586240
Gerrit-PatchSet: 3
Gerrit-Owner: Gerrit Bot <letsus...@gmail.com>
Gerrit-Reviewer: Dmitri Shuralyov <dmit...@golang.org>
Gerrit-Reviewer: Dmitri Shuralyov <dmit...@google.com>
Gerrit-Reviewer: Gopher Robot <go...@golang.org>
Gerrit-Reviewer: Ian Lance Taylor <ia...@google.com>
Gerrit-Reviewer: Keith Randall <k...@golang.org>
Gerrit-CC: kyose chang <sev...@gmail.com>
open
diffy
satisfied_requirement
Reply all
Reply to author
Forward
0 new messages