Skip to content

Commit

Permalink
Use raw concat instead of fmt.Sprintf for fullyQualifiedName (#124)
Browse files Browse the repository at this point in the history
  • Loading branch information
mway authored Jan 21, 2020
1 parent f874b52 commit 9709da0
Showing 1 changed file with 1 addition and 2 deletions.
3 changes: 1 addition & 2 deletions scope.go
Original file line number Diff line number Diff line change
Expand Up @@ -21,7 +21,6 @@
package tally

import (
"fmt"
"io"
"sync"
"time"
Expand Down Expand Up @@ -575,7 +574,7 @@ func (s *scope) fullyQualifiedName(name string) string {
// output we're creating is a concatenation of the sanitized inputs.
// If we change the concatenation to involve other inputs or characters,
// we'll need to sanitize them too.
return fmt.Sprintf("%s%s%s", s.prefix, s.separator, name)
return s.prefix + s.separator + name
}

func (s *scope) copyAndSanitizeMap(tags map[string]string) map[string]string {
Expand Down

0 comments on commit 9709da0

Please sign in to comment.