tree 0ed989a5abeaffe741fa9b4ccdb99f78baf72ba8
parent a28edbfca276307b228eb4b154bc2d137a3cba4a
author Daniel Martí <mvdan@mvdan.cc> 1601150437 +0100
committer Daniel Martí <mvdan@mvdan.cc> 1601376775 +0000

cmd/go: fix doc math for build cache hashing

The function takes five 24-bit chunks from the hash, resulting in 120
bits. When base-64 encoded, this results in a 20-byte output string,
which is confirmed by "var dst [chunks * 4]byte".

It seems like the documented math could have been written for a previous
implementation with shorter hashes, using 4 chunks instead of 5, as then
the math checks out.

Since this code has been working correctly for over three years, let's
fix the documentation to reflect the code.

Change-Id: I9e908e6bafb5dcc1e3c23915e2b6c8843ed444d6
Reviewed-on: https://go-review.googlesource.com/c/go/+/257646
Trust: Daniel Martí <mvdan@mvdan.cc>
Run-TryBot: Daniel Martí <mvdan@mvdan.cc>
TryBot-Result: Go Bot <gobot@golang.org>
Reviewed-by: Bryan C. Mills <bcmills@google.com>