WIP

Author: solnic

sentry-ruby/lib/sentry/backtrace.rb

@@ -32,36 +32,16 @@ def self.parse(backtrace, project_root, app_dirs_pattern, &backtrace_cleanup_cal
     # @see https://github.com/rails/rails/pull/49095 for more context.
     if Thread.respond_to?(:each_caller_location)
       def self.source_location(backtrace_cleaner)
-        cache = line_cache.fetch_or_store(backtrace_cleaner.__id__) { Concurrent::Map.new }
-
         Thread.each_caller_location do |location|
-          frame_key = [location.absolute_path, location.lineno]
-          cached_value = cache[frame_key]
-
-          next if cached_value == :skip
-
-          if cached_value
-            return cached_value
-          else
-            cleaned_frame = backtrace_cleaner.clean_frame(location)
-
-            if cleaned_frame
-              line = Line.from_source_location(location)
-              cache[frame_key] = line
+          cleaned_frame = backtrace_cleaner.clean_frame(location)
 
-              return line
-            else
-              cache[frame_key] = :skip
+          next unless cleaned_frame
 
-              next
-            end
+          if cleaned_frame
+            return Line.from_source_location(location)
           end
         end
       end
-
-      def self.line_cache
-        @line_cache ||= Concurrent::Map.new
-      end
     else
       # Since Sentry is mostly used in production, we don't want to fallback
       # to the slower implementation and adds potentially big overhead to the
@@ -71,7 +51,6 @@ def self.source_location(*)
       end
     end
 
-
     def initialize(lines)
       @lines = lines
     end