Optimize removeDuplicates function

- Use struct{} instead of bool in the map to reduce memory usage.
- Combine declaration and initialization of the result slice.
- Initialize result slice with a capacity equal to the input slice
  length to reduce allocations.

These changes improve memory efficiency and make the code more concise.

Signed-off-by: Chmouel Boudjnah <[email protected]>

Author: chmouel

pkg/changedfiles/changedfiles.go

@@ -8,12 +8,13 @@ type ChangedFiles struct {
 	Renamed  []string
 }
 
+// removeDuplicates removes duplicates from a slice of strings.
 func removeDuplicates(s []string) []string {
-	holdit := make(map[string]bool)
-	var result []string
+	holdit := make(map[string]struct{})
+	result := make([]string, 0, len(s))
 	for _, str := range s {
 		if _, ok := holdit[str]; !ok {
-			holdit[str] = true
+			holdit[str] = struct{}{}
 			result = append(result, str)
 		}
 	}