Avoid creating unnecessary strings when key is already in correct format

9313ba19 · Kevin Atkinson · 971a83c4 · 9313ba19
Commit 9313ba19 authored Nov 17, 2016 by Kevin Atkinson
Hide whitespace changes
Inline Side-by-side

Showing with 8 additions and 1 deletion

key.go key.go +8 -1

No files found.
--- a/key.go
+++ b/key.go
@@ -65,7 +65,14 @@ func KeyWithNamespaces(ns []string) Key {

 // Clean up a Key, using path.Clean.
 func (k *Key) Clean() {
-	k.string = path.Clean("/" + k.string)
+	switch {
+	case len(k.string) == 0:
+		k.string = "/"
+	case k.string[0] == '/':
+		k.string = path.Clean(k.string)
+	default:
+		k.string = path.Clean("/" + k.string)
+	}
 }

 // Strings is the string value of Key