Skip to content
Browse files

refactor Range#include? for range value

  • Loading branch information...
1 parent bec7cf2 commit 3f642c9d1834a4ef586461648acc7c849bebad70 @nashby nashby committed Dec 26, 2011
Showing with 1 addition and 3 deletions.
  1. +1 −3 activesupport/lib/active_support/core_ext/range/include_range.rb
View
4 activesupport/lib/active_support/core_ext/range/include_range.rb
@@ -9,9 +9,7 @@ class Range
# (5..9).include?(11) # => false
def include_with_range?(value)
if value.is_a?(::Range)
- operator = exclude_end? ? :< : :<=
- end_value = value.exclude_end? ? last.succ : last
- include_without_range?(value.first) && (value.last <=> end_value).send(operator, 0)
+ min <= value.min && max >= value.max
else
include_without_range?(value)
end

0 comments on commit 3f642c9

Please sign in to comment.
Something went wrong with that request. Please try again.