chore: Replace fmt.Printf calls with warning logs (#1305)

Co-authored-by: wwared <[email protected]>
Consensys · Nov 15, 2024 · 525615c · 525615c
1 parent c975bac
commit 525615c
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 2 deletions.
diff --git a/frontend/schema/schema.go b/frontend/schema/schema.go
@@ -22,6 +22,8 @@ import (
 	"reflect"
 	"strconv"
 	"strings"
+
+	"github.com/consensys/gnark/logger"
 )
 
 // Schema represents the structure of a gnark circuit (/ witness)
@@ -322,7 +324,8 @@ func parse(r []Field, input interface{}, target reflect.Type, parentFullName, pa
 	if tValue.Kind() == reflect.Slice || tValue.Kind() == reflect.Array {
 		if tValue.Len() == 0 {
 			if reflect.SliceOf(target) == tValue.Type() {
-				fmt.Printf("ignoring uninitialized slice: %s %s\n", parentGoName, reflect.SliceOf(target).String())
+				log := logger.Logger()
+				log.Warn().Str("slice name", parentGoName).Str("slice type", reflect.SliceOf(target).String()).Msg("ignoring uninitialized slice")
 			}
 			return r, nil
 		}

diff --git a/frontend/schema/walk.go b/frontend/schema/walk.go
@@ -7,6 +7,7 @@ import (
 	"strings"
 
 	"github.com/consensys/gnark/frontend/schema/internal/reflectwalk"
+	"github.com/consensys/gnark/logger"
 )
 
 // Walk walks through the provided object and stops when it encounters objects of type tLeaf
@@ -82,7 +83,8 @@ func (w *walker) Pointer(value reflect.Value) error {
 func (w *walker) Slice(value reflect.Value) error {
 	if value.Type() == w.targetSlice {
 		if value.Len() == 0 {
-			fmt.Printf("ignoring uninitialized slice: %s %s\n", w.name(), reflect.SliceOf(w.target).String())
+			log := logger.Logger()
+			log.Warn().Str("slice name", w.name()).Str("slice type", reflect.SliceOf(w.target).String()).Msg("ignoring uninitialized slice")
 			return nil
 		}
 		return w.handleLeaves(value)