Merge pull request #7543 from heyitsanthony/fix-timeseries

*: fix gosimple warning for fmt.Sprintf("%s")
etcd-io · Mar 20, 2017 · 3a1368d · 3a1368d
2 parents 53ca03b + ae7b4ee
commit 3a1368d
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 4 deletions.
diff --git a/etcdserver/membership/cluster.go b/etcdserver/membership/cluster.go
@@ -178,7 +178,7 @@ func (c *RaftCluster) String() string {
 	fmt.Fprintf(b, "Members:[%s] ", strings.Join(ms, " "))
 	var ids []string
 	for id := range c.removed {
-		ids = append(ids, fmt.Sprintf("%s", id))
+		ids = append(ids, id.String())
 	}
 	fmt.Fprintf(b, "RemovedMemberIDs:[%s]}", strings.Join(ids, " "))
 	return b.String()

diff --git a/pkg/report/timeseries.go b/pkg/report/timeseries.go
@@ -128,9 +128,9 @@ func (ts TimeSeries) String() string {
 	for i := range ts {
 		row := []string{
 			fmt.Sprintf("%d", ts[i].Timestamp),
-			fmt.Sprintf("%s", ts[i].MinLatency),
-			fmt.Sprintf("%s", ts[i].AvgLatency),
-			fmt.Sprintf("%s", ts[i].MaxLatency),
+			ts[i].MinLatency.String(),
+			ts[i].AvgLatency.String(),
+			ts[i].MaxLatency.String(),
 			fmt.Sprintf("%d", ts[i].ThroughPut),
 		}
 		rows = append(rows, row)