diff options
| author | Viktar Basharymau <viktar.basharymau@thehamon.com> | 2014-05-22 17:44:25 +0300 |
|---|---|---|
| committer | Viktar Basharymau <viktar.basharymau@thehamon.com> | 2014-05-22 17:44:25 +0300 |
| commit | eba8b70d4ec7239694b026bef3c07e5ad82efe69 (patch) | |
| tree | b0d3b08c3fe77265cb266c45cbb80033656a71a4 | |
| parent | 66dd216705c9401cfa559a52910df80bd9c7808b (diff) | |
| download | rails-eba8b70d4ec7239694b026bef3c07e5ad82efe69.tar.gz rails-eba8b70d4ec7239694b026bef3c07e5ad82efe69.tar.bz2 rails-eba8b70d4ec7239694b026bef3c07e5ad82efe69.zip | |
Use `break` instead of `next` in AD::Journey::Formatter#match_route
The array is sorted in descending order, so there is no point in
iterating further if we met a negative item - all the rest will be
negative too.
| -rw-r--r-- | actionpack/lib/action_dispatch/journey/formatter.rb | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/actionpack/lib/action_dispatch/journey/formatter.rb b/actionpack/lib/action_dispatch/journey/formatter.rb index 7eaf8e49ce..4ed74225f7 100644 --- a/actionpack/lib/action_dispatch/journey/formatter.rb +++ b/actionpack/lib/action_dispatch/journey/formatter.rb @@ -85,7 +85,7 @@ module ActionDispatch hash = routes.group_by { |_, r| r.score(options) } hash.keys.sort.reverse_each do |score| - next if score < 0 + break if score < 0 hash[score].sort_by { |i, _| i }.each do |_, route| yield route |