fix(encoder): missing error details on calculateDataSize, add more de…

…tails for verbosity (#22)
muktihari · Nov 30, 2023 · 63857e9 · 63857e9
1 parent 952c9f8
commit 63857e9
Showing 1 changed file with 8 additions and 5 deletions.
diff --git a/encoder/encoder.go b/encoder/encoder.go
@@ -324,8 +324,10 @@ func (e *Encoder) updateHeader(header *proto.FileHeader) error {
 func (e *Encoder) calculateDataSize(fit *proto.Fit) error {
 	n := e.n
 	for i := range fit.Messages { // calculating messages actual size
-		if err := e.encodeMessage(io.Discard, &fit.Messages[i]); err != nil {
-			return err
+		mesg := &fit.Messages[i]
+		if err := e.encodeMessage(io.Discard, mesg); err != nil {
+			return fmt.Errorf("encode failed: at byte pos: %d, message index: %d, num: %d (%s): %w",
+				e.n, i, mesg.Num, mesg.Num.String(), err)
 		}
 	}
 
@@ -376,9 +378,10 @@ func (e *Encoder) encodeMessages(messages []proto.Message) error {
 	}
 
 	for i := range messages {
-		if err := e.encodeMessage(e.w, &messages[i]); err != nil {
-			return fmt.Errorf("encode failed: at byte pos: %d, message index: %d, num: %d: %w",
-				e.n, i, messages[i].Num, err)
+		mesg := &messages[i]
+		if err := e.encodeMessage(e.w, mesg); err != nil {
+			return fmt.Errorf("encode failed: at byte pos: %d, message index: %d, num: %d (%s): %w",
+				e.n, i, mesg.Num, mesg.Num.String(), err)
 		}
 	}