Skip to content

Commit 51e532c

Browse files
committed
name_filters is always the same inside the instance
1 parent 18384bc commit 51e532c

1 file changed

Lines changed: 4 additions & 5 deletions

File tree

lib/benchmark_suite.rb

Lines changed: 4 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -127,24 +127,23 @@ def bench_file_grouping
127127
bench_file_grouping = {}
128128

129129
# Get the list of benchmark files/directories matching name filters
130-
filter = benchmark_filter(categories: categories, name_filters: name_filters)
130+
filter = benchmark_filter(categories: categories)
131131
bench_file_grouping[bench_dir] = filtered_bench_entries(bench_dir, filter)
132132

133133
if benchmark_ractor_directory?
134134
# We ignore the category filter here because everything in the
135135
# benchmarks-ractor directory should be included when we're benchmarking the
136136
# Ractor category
137-
ractor_filter = benchmark_filter(categories: [], name_filters: name_filters)
137+
ractor_filter = benchmark_filter(categories: [])
138138
bench_file_grouping[ractor_bench_dir] = filtered_bench_entries(ractor_bench_dir, ractor_filter)
139139
end
140140

141141
bench_file_grouping
142142
end
143143

144-
def benchmark_filter(categories:, name_filters:)
144+
def benchmark_filter(categories:)
145145
@benchmark_filter ||= {}
146-
key = [categories, name_filters]
147-
@benchmark_filter[key] ||= BenchmarkFilter.new(
146+
@benchmark_filter[categories] ||= BenchmarkFilter.new(
148147
categories: categories,
149148
name_filters: name_filters,
150149
metadata: benchmarks_metadata

0 commit comments

Comments
 (0)