meineerde/rackstash
1
0
Fork 0
mirror of https://github.com/meineerde/rackstash.git synced 2026-02-01 01:37:12 +00:00
Code Issues Releases Activity

Use Range@cover? instead of Range#include? in FilterChain for slightly better performance

Browse Source
This commit is contained in:
Holger Just 2017-08-05 15:05:04 +02:00
parent 77675f65a6
commit 7ba2010412
1 changed files with 3 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

6
lib/rackstash/filter_chain.rb
View File

@ -67,7 +67,7 @@ module Rackstash
synchronize do
id = index_at(index)
unless id && (0..@filters.size).include?(id)
unless id && (0..@filters.size).cover?(id)
raise ArgumentError, "Cannot insert at index #{index.inspect}"
end
@ -185,7 +185,7 @@ module Rackstash
synchronize do
id = index_at(index)
unless id && (0...@filters.size).include?(id)
unless id && (0...@filters.size).cover?(id)
raise ArgumentError, "No such filter to insert after: #{index.inspect}"
end
@ -215,7 +215,7 @@ module Rackstash
synchronize do
id = index_at(index)
unless id && (0...@filters.size).include?(id)
unless id && (0...@filters.size).cover?(id)
raise ArgumentError, "No such filter to insert before: #{index.inspect}"
end