reader: fix Skip(n) where n >= buffer size

The (*Reader).Skip() code did not handle skip sizes above the buffer size correctly, because the inner reader loop only looped while 'r.buffered() < n', and that condition is obviously violated when the reader successfully buffers more bytes than are to be skipped. Refactor this method to be a little shorter and hopefully much clearer (and more correct) and add test coverage that exercises this code better.
2026-10-22 19:09:29 +00:00 · 2020-11-24 00:41:02 +00:00
7 changed files with 84 additions and 426 deletions
--- a/README.md
+++ b/README.md
--- a/go.mod
+++ b/go.mod
--- a/reader.go
+++ b/reader.go
--- a/reader_test.go
+++ b/reader_test.go
--- a/writer_appengine.go
+++ b/writer_appengine.go
--- a/writer_tinygo.go
+++ b/writer_tinygo.go
--- a/writer_unsafe.go
+++ b/writer_unsafe.go