S1003: should use strings.ContainsRune(" \t\r\n", rune(c)) instead (g…

…osimple)
heroku · Sep 9, 2019 · 7f09935 · 7f09935
1 parent c7b441c
commit 7f09935
Showing 1 changed file with 2 additions and 2 deletions.
diff --git a/registry/authchallenge.go b/registry/authchallenge.go
@@ -44,8 +44,8 @@ func init() {
 		var t octetType
 		isCtl := c <= 31 || c == 127
 		isChar := 0 <= c && c <= 127
-		isSeparator := strings.IndexRune(" \t\"(),/:;<=>?@[]\\{}", rune(c)) >= 0
-		if strings.IndexRune(" \t\r\n", rune(c)) >= 0 {
+		isSeparator := strings.ContainsRune(" \t\"(),/:;<=>?@[]\\{}", rune(c))
+		if strings.ContainsRune(" \t\r\n", rune(c)) {
 			t |= isSpace
 		}
 		if isChar && !isCtl && !isSeparator {