diff options
| author | Matthew Dempsky <mdempsky@google.com> | 2017-09-19 14:28:03 -0700 |
|---|---|---|
| committer | Matthew Dempsky <mdempsky@google.com> | 2017-09-20 17:47:26 +0000 |
| commit | e06a64a4768445f2fcea1cb6b5a04abe0c4d9d12 (patch) | |
| tree | bde00cb4598ea2b5cd52c78e5b3f8ad837d3176e /src/cmd/compile/internal/syntax/source.go | |
| parent | a53e853964cc9220ebc4b35aeb81a382939fb479 (diff) | |
| download | go-git-e06a64a4768445f2fcea1cb6b5a04abe0c4d9d12.tar.gz | |
cmd/compile/internal/syntax: fix source buffer refilling
The previous code seems to have an off-by-1 in it somewhere, the
consequence being that we didn't properly preserve all of the old
buffer contents that we intended to.
After spending a while looking at the existing window-shifting logic,
I wasn't able to understand exactly how it was supposed to work or
where the issue was, so I rewrote it to be (at least IMO) more
obviously correct.
Fixes #21938.
Change-Id: I1ed7bbc1e1751a52ab5f7cf0411ae289586dc345
Reviewed-on: https://go-review.googlesource.com/64830
Run-TryBot: Matthew Dempsky <mdempsky@google.com>
TryBot-Result: Gobot Gobot <gobot@golang.org>
Reviewed-by: Robert Griesemer <gri@golang.org>
Diffstat (limited to 'src/cmd/compile/internal/syntax/source.go')
| -rw-r--r-- | src/cmd/compile/internal/syntax/source.go | 11 |
1 files changed, 6 insertions, 5 deletions
diff --git a/src/cmd/compile/internal/syntax/source.go b/src/cmd/compile/internal/syntax/source.go index 93547213c0..4e3551225a 100644 --- a/src/cmd/compile/internal/syntax/source.go +++ b/src/cmd/compile/internal/syntax/source.go @@ -164,11 +164,12 @@ func (s *source) fill() { s.lit = append(s.lit, s.buf[s.suf:s.r0]...) s.suf = 1 // == s.r0 after slide below } - s.offs += s.r0 - 1 - r := s.r - s.r0 + 1 // last read char plus one byte - s.w = r + copy(s.buf[r:], s.buf[s.r:s.w]) - s.r = r - s.r0 = 1 + n := s.r0 - 1 + copy(s.buf[:], s.buf[n:s.w]) + s.offs += n + s.r0 = 1 // eqv: s.r0 -= n + s.r -= n + s.w -= n } // read more data: try a limited number of times |