Permalink
Browse files

Merge pull request #17639 from gsamokovarov/faster-exception-wrapper-…

…traces

Don't double check trace origin in ExceptionWrapper#traces
  • Loading branch information...
guilleiguaran committed Nov 16, 2014
2 parents 28dfc6a + ff19027 commit 7830bb0d9d01b125755e226ffdca8a0f2cc7d00f
Showing with 6 additions and 2 deletions.
  1. +6 −2 actionpack/lib/action_dispatch/middleware/exception_wrapper.rb
@@ -65,8 +65,12 @@ def traces
full_trace.each_with_index do |trace, idx|
trace_with_id = { id: idx, trace: trace }
appplication_trace_with_ids << trace_with_id if application_trace.include?(trace)
framework_trace_with_ids << trace_with_id if framework_trace.include?(trace)
if application_trace.include?(trace)
appplication_trace_with_ids << trace_with_id
else
framework_trace_with_ids << trace_with_id
end
full_trace_with_ids << trace_with_id
end

0 comments on commit 7830bb0

Please sign in to comment.