diff options
| author | Robert Griesemer <gri@golang.org> | 2020-08-21 10:02:44 -0700 |
|---|---|---|
| committer | Robert Griesemer <gri@golang.org> | 2020-08-25 16:06:28 +0000 |
| commit | 27136419d4784d3f81af3ec1faf0962a60c69b7e (patch) | |
| tree | 8f59eba0180c75d3a9c2cd751b3335fa78f9acbd /src/encoding | |
| parent | bca0b44629f1317cc177072560d9a1486620e48f (diff) | |
| download | go-27136419d4784d3f81af3ec1faf0962a60c69b7e.tar.gz go-27136419d4784d3f81af3ec1faf0962a60c69b7e.zip | |
encoding/binary: replace constant literals with named constant (cleanup)
Follow-up on https://golang.org/cl/247120.
Brought to my attention by Luke McCoy.
Change-Id: I4530c96fb164d23b0ce5311f2cecb1964f2dea74
Reviewed-on: https://go-review.googlesource.com/c/go/+/249837
Reviewed-by: Katie Hockman <katie@golang.org>
Diffstat (limited to 'src/encoding')
| -rw-r--r-- | src/encoding/binary/varint.go | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/src/encoding/binary/varint.go b/src/encoding/binary/varint.go index 38af61075c..1fa325dec7 100644 --- a/src/encoding/binary/varint.go +++ b/src/encoding/binary/varint.go @@ -62,7 +62,7 @@ func Uvarint(buf []byte) (uint64, int) { var s uint for i, b := range buf { if b < 0x80 { - if i > 9 || i == 9 && b > 1 { + if i >= MaxVarintLen64 || i == MaxVarintLen64-1 && b > 1 { return 0, -(i + 1) // overflow } return x | uint64(b)<<s, i + 1 @@ -112,7 +112,7 @@ func ReadUvarint(r io.ByteReader) (uint64, error) { return x, err } if b < 0x80 { - if i == 9 && b > 1 { + if i == MaxVarintLen64-1 && b > 1 { return x, overflow } return x | uint64(b)<<s, nil |