Change isSimple to allow >= 0x21 and <= 0x7e

Author: vmihailenco

cmdutil/cmd.go

@@ -146,77 +146,5 @@ func isSimple(b []byte) bool {
 }
 
 func isSimpleByte(c byte) bool {
-	return simple[c]
-}
-
-var simple = [256]bool{
-	'-': true,
-	'+': true,
-	'_': true,
-	':': true,
-
-	'0': true,
-	'1': true,
-	'2': true,
-	'3': true,
-	'4': true,
-	'5': true,
-	'6': true,
-	'7': true,
-	'8': true,
-	'9': true,
-
-	'a': true,
-	'b': true,
-	'c': true,
-	'd': true,
-	'e': true,
-	'f': true,
-	'g': true,
-	'h': true,
-	'i': true,
-	'j': true,
-	'k': true,
-	'l': true,
-	'm': true,
-	'n': true,
-	'o': true,
-	'p': true,
-	'q': true,
-	'r': true,
-	's': true,
-	't': true,
-	'u': true,
-	'v': true,
-	'w': true,
-	'x': true,
-	'y': true,
-	'z': true,
-
-	'A': true,
-	'B': true,
-	'C': true,
-	'D': true,
-	'E': true,
-	'F': true,
-	'G': true,
-	'H': true,
-	'I': true,
-	'J': true,
-	'K': true,
-	'L': true,
-	'M': true,
-	'N': true,
-	'O': true,
-	'P': true,
-	'Q': true,
-	'R': true,
-	'S': true,
-	'T': true,
-	'U': true,
-	'V': true,
-	'W': true,
-	'X': true,
-	'Y': true,
-	'Z': true,
+	return c >= 0x21 && c <= 0x7e
 }

cmdutil/cmd_test.go

@@ -22,7 +22,9 @@ var _ = Describe("AppendArg", func() {
 
 		Entry("", "-inf", "-inf"),
 		Entry("", "+inf", "+inf"),
+		Entry("", "foo.bar", "foo.bar"),
 		Entry("", "foo:bar", "foo:bar"),
+		Entry("", "foo{bar}", "foo{bar}"),
 		Entry("", "foo-123_BAR", "foo-123_BAR"),
 		Entry("", "foo\nbar", "666f6f0a626172"),
 		Entry("", "\000", "00"),