Sync counters on read/write error (incl io.EOF)

.github/workflows/release-assets.yml

@@ -34,6 +34,7 @@ jobs:
         uses: actions/checkout@v3
       - name: Build artifacts
         run: |
+          git fetch -t
           make release-assets
       - name: Upload linux_amd64.tar.gz
         if: hashFiles('linux_amd64.tar.gz') != ''

cmd/catp/catp/app.go

@@ -285,8 +285,6 @@
 		r.readFile(rd, out)
 	}
 
-	cr.Close()
-
 	if r.parallel <= 1 {
 		r.pr.Stop()
 	}
@@ -391,7 +389,8 @@
 	ver := flag.Bool("version", false, "print version and exit")
 
 	flag.Usage = func() {
-		fmt.Println("catp", version.Info().Version+",", version.Info().GoVersion, strings.Join(versionExtra, " "))
+		fmt.Println("catp", version.Module("github.com/bool64/progress").Version+",",
+			version.Info().GoVersion, strings.Join(versionExtra, " "))
 		fmt.Println()
 		fmt.Println("catp prints contents of files to STDOUT or dir/file output, \n" +
 			"while printing current progress status to STDERR. \n" +

progress.go

@@ -277,10 +277,10 @@
 	cr.bytes = bytes
 }
 
-func (cr *sharedCounters) count(n int, p []byte) {
+func (cr *sharedCounters) count(n int, p []byte, err error) {
 	cr.localBytes += int64(n)
 
-	if cr.localBytes > 100000 && cr.bytes != nil {
+	if (err != nil || cr.localBytes > 100000) && cr.bytes != nil {
 		atomic.AddInt64(cr.bytes, cr.localBytes)
 		cr.localBytes = 0
 	}
@@ -292,19 +292,19 @@
 	for i := 0; i < n; i++ {
 		if p[i] == '\n' {
 			cr.localLines++
-
-			if cr.localLines > 1000 {
-				atomic.AddInt64(cr.lines, cr.localLines)
-				cr.localLines = 0
-			}
 		}
 	}
+
+	if err != nil || cr.localLines > 1000 {
+		atomic.AddInt64(cr.lines, cr.localLines)
+		cr.localLines = 0
+	}
 }
 
 // Read reads and counts bytes.
 func (cr *CountingReader) Read(p []byte) (n int, err error) {
 	n, err = cr.Reader.Read(p)
-	cr.count(n, p)
+	cr.count(n, p, err)
 
 	return n, err
 }
@@ -349,7 +349,7 @@
 // Write writes and counts bytes.
 func (cr *CountingWriter) Write(p []byte) (n int, err error) {
 	n, err = cr.Writer.Write(p)
-	cr.count(n, p)
+	cr.count(n, p, err)
 
 	return n, err
 }