Merge pull request #49 from theckman/reset_buffer

Reset the buffer when constructing the spinner
theckman · Dec 15, 2021 · 5a67d42 · 5a67d42
2 parents 4b45330 + d19965d
commit 5a67d42
Show file tree

Hide file tree

Showing 2 changed files with 8 additions and 1 deletion.
diff --git a/spinner.go b/spinner.go
@@ -256,8 +256,11 @@ func New(cfg Config) (*Spinner, error) {
 		cfg.NotTTY = true
 	}
 
+	buf := bytes.NewBuffer(make([]byte, 2048))
+	buf.Reset()
+
 	s := &Spinner{
-		buffer:            bytes.NewBuffer(make([]byte, 2048)),
+		buffer:            buf,
 		mu:                &sync.Mutex{},
 		frequency:         cfg.Frequency,
 		status:            uint32Ptr(0),

diff --git a/spinner_test.go b/spinner_test.go
@@ -132,6 +132,10 @@ func TestNew(t *testing.T) {
 				t.Fatal("spinner is nil")
 			}
 
+			if n := spinner.buffer.Len(); n != 0 {
+				t.Fatalf("spinner.buffer.Len() = %d, want 0", n)
+			}
+
 			if spinner.colorAll != tt.cfg.ColorAll {
 				t.Fatalf("spinner.colorAll = %t, want %t", spinner.colorAll, tt.cfg.ColorAll)
 			}