all: remove unnecessary string([]byte) conversion in fmt.Sprintf with %s
Change-Id: I64aab811cbcbfa410817894f1cd1d83f88f27bf6
Reviewed-on: https://go-review.googlesource.com/c/protobuf/+/365874
Reviewed-by: Damien Neil <dneil@google.com>
Trust: Damien Neil <dneil@google.com>
Trust: Emmanuel Odeke <emmanuel@orijtech.com>
diff --git a/internal/encoding/text/decode.go b/internal/encoding/text/decode.go
index eb10ea1..3780377 100644
--- a/internal/encoding/text/decode.go
+++ b/internal/encoding/text/decode.go
@@ -381,7 +381,7 @@
case '[':
return ListOpen, ']'
}
- panic(fmt.Sprintf("Decoder: openStack contains invalid byte %s", string(openCh)))
+ panic(fmt.Sprintf("Decoder: openStack contains invalid byte %c", openCh))
}
func (d *Decoder) pushOpenStack(ch byte) {
diff --git a/testing/prototest/message.go b/testing/prototest/message.go
index e495628..c104605 100644
--- a/testing/prototest/message.go
+++ b/testing/prototest/message.go
@@ -643,7 +643,7 @@
if err != nil {
return fmt.Sprintf("<%v>", err)
}
- return fmt.Sprintf("%v{%v}", v.Descriptor().FullName(), string(b))
+ return fmt.Sprintf("%v{%s}", v.Descriptor().FullName(), b)
case string:
return fmt.Sprintf("%q", v)
default:
