Skip to content

Commit 97d28c9

Browse files
Deduplicate results just once
Instead of checking for uniqueness for every spec.
1 parent 9d7bd17 commit 97d28c9

File tree

1 file changed

+3
-10
lines changed

1 file changed

+3
-10
lines changed

bundler/lib/bundler/index.rb

Lines changed: 3 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -58,19 +58,12 @@ def search_all(name)
5858
# about, returning all of the results.
5959
def search(query)
6060
results = local_search(query)
61-
62-
seen = results.map(&:full_name).uniq unless @sources.empty?
61+
return results unless @sources.any?
6362

6463
@sources.each do |source|
65-
source.search(query).each do |spec|
66-
next if seen.include?(spec.full_name)
67-
68-
seen << spec.full_name
69-
results << spec
70-
end
64+
results.concat(source.search(query))
7165
end
72-
73-
results
66+
results.uniq(&:full_name)
7467
end
7568

7669
def local_search(query)

0 commit comments

Comments
 (0)