refactor(serverHandler/responseData): prevent memory reallocation

Author: marjune163

src/serverHandler/responseData.go

@@ -72,8 +72,10 @@ func isSlash(c rune) bool {
 }
 
 func getPathEntries(path string, tailSlash bool) []pathEntry {
-	paths := []string{"/"}
-	paths = append(paths, strings.FieldsFunc(path, isSlash)...)
+	restPaths := strings.FieldsFunc(path, isSlash)
+	paths := make([]string, 1, len(restPaths)+1)
+	paths[0] = "/"
+	paths = append(paths, restPaths...)
 
 	displayPathsCount := len(paths)
 

src/serverHandler/responseData_test.go

@@ -0,0 +1,41 @@
+package serverHandler
+
+import "testing"
+
+func TestGetPathEntries(t *testing.T) {
+	var result []pathEntry
+
+	result = getPathEntries("/a/b/c", false)
+	if len(result) != 4 {
+		t.Error(len(result))
+	}
+	if result[0].Path != "../../" || result[0].Name != "/" {
+		t.Error(result[0])
+	}
+	if result[1].Path != "../" || result[1].Name != "a" {
+		t.Error(result[1])
+	}
+	if result[2].Path != "./" || result[2].Name != "b" {
+		t.Error(result[2])
+	}
+	if result[3].Path != "./c/" || result[3].Name != "c" {
+		t.Error(result[3])
+	}
+
+	result = getPathEntries("/a/b/c", true)
+	if len(result) != 4 {
+		t.Error(len(result))
+	}
+	if result[0].Path != "../../../" || result[0].Name != "/" {
+		t.Error(result[0])
+	}
+	if result[1].Path != "../../" || result[1].Name != "a" {
+		t.Error(result[1])
+	}
+	if result[2].Path != "../" || result[2].Name != "b" {
+		t.Error(result[2])
+	}
+	if result[3].Path != "./" || result[3].Name != "c" {
+		t.Error(result[3])
+	}
+}