fix: Always call Finish() in Reader

mrkbac · web-flow · commit 9d2ae6b6797f · 2025-05-16T22:03:24.000+02:00
diff --git a/progressbar.go b/progressbar.go
@@ -1418,7 +1418,7 @@ func (r *Reader) Read(p []byte) (n int, err error) {
 // Close the reader when it implements io.Closer
 func (r *Reader) Close() (err error) {
 	if closer, ok := r.Reader.(io.Closer); ok {
-		return closer.Close()
+		err = closer.Close()
 	}
 	r.bar.Finish()
 	return

Original file line number	Diff line number	Diff line change
`@@ -1418,7 +1418,7 @@ func (r *Reader) Read(p []byte) (n int, err error) {`
`1418`	`1418`	`// Close the reader when it implements io.Closer`
`1419`	`1419`	`func (r *Reader) Close() (err error) {`
`1420`	`1420`	`if closer, ok := r.Reader.(io.Closer); ok {`
`1421`		`- return closer.Close()`
	`1421`	`+ err = closer.Close()`
`1422`	`1422`	`}`
`1423`	`1423`	`r.bar.Finish()`
`1424`	`1424`	`return`