Factored a method in library.List

Author: cmaglie

arduino/libraries/librarieslist.go

@@ -39,6 +39,20 @@ func (list *List) Add(libs ...*Library) {
 	}
 }
 
+// Remove removes the given library from the list
+func (list *List) Remove(libraryToRemove *Library) {
+	j := 0
+	for i, lib := range *list {
+		if lib != libraryToRemove {
+			if i != j {
+				(*list)[j] = (*list)[i]
+			}
+			j++
+		}
+	}
+	*list = (*list)[:j]
+}
+
 // FindByName returns the first library in the list that match
 // the specified name or nil if not found
 func (list *List) FindByName(name string) *Library {

legacy/builder/resolve_library.go

@@ -58,20 +58,11 @@ func ResolveLibrary(ctx *types.Context, header string) *libraries.Library {
 		}
 	}
 
+	candidates.Remove(selected)
 	ctx.LibrariesResolutionResults[header] = types.LibraryResolutionResult{
 		Library:          selected,
-		NotUsedLibraries: filterOutLibraryFrom(candidates, selected),
+		NotUsedLibraries: candidates,
 	}
 
 	return selected
 }
-
-func filterOutLibraryFrom(libs libraries.List, libraryToRemove *libraries.Library) libraries.List {
-	filteredOutLibraries := []*libraries.Library{}
-	for _, lib := range libs {
-		if lib != libraryToRemove {
-			filteredOutLibraries = append(filteredOutLibraries, lib)
-		}
-	}
-	return filteredOutLibraries
-}