chore: use strings.builder instead of byte buffer pool (#81)

Signed-off-by: Timon Wong <[email protected]>

Author: timonwong

internal/bytebufferpool/pool.go

@@ -6,11 +6,10 @@ import (
 	"go/printer"
 	"go/token"
 	"go/types"
+	"strings"
 	"unicode/utf8"
 
 	"golang.org/x/tools/go/analysis"
-
-	"github.com/timonwong/loggercheck/internal/bytebufferpool"
 )
 
 const (
@@ -31,9 +30,7 @@ func extractValueFromStringArg(pass *analysis.Pass, arg ast.Expr) (value string,
 func renderNodeEllipsis(fset *token.FileSet, v interface{}) string {
 	const maxLen = 20
 
-	buf := bytebufferpool.Get()
-	defer bytebufferpool.Put(buf)
-
+	buf := &strings.Builder{}
 	_ = printer.Fprint(buf, fset, v)
 	s := buf.String()
 	if utf8.RuneCountInString(s) > maxLen {

internal/checkers/common.go

@@ -7,8 +7,6 @@ import (
 	"go/types"
 	"io"
 	"strings"
-
-	"github.com/timonwong/loggercheck/internal/bytebufferpool"
 )
 
 var ErrInvalidRule = errors.New("invalid rule format")
@@ -44,8 +42,7 @@ func (rs *Ruleset) Match(fn *types.Func) bool {
 }
 
 func receiverTypeOf(recvType types.Type) string {
-	buf := bytebufferpool.Get()
-	defer bytebufferpool.Put(buf)
+	buf := &strings.Builder{}
 
 	var recvNamed *types.Named
 	switch recvType := recvType.(type) {