diff options
| author | schneems <[email protected]> | 2023-04-14 13:46:28 -0500 |
|---|---|---|
| committer | Hiroshi SHIBATA <[email protected]> | 2023-05-23 10:05:47 +0900 |
| commit | aaf815c626816ccca227cb1f8f2a9b58f18f677a (patch) | |
| tree | df1851edb5ed016a1b41124b851b3bf7a6daef40 /lib/syntax_suggest/capture_code_context.rb | |
| parent | b8bf0a52723322e469e94a4cb96286fd48e754ab (diff) | |
[ruby/syntax_suggest] Refactor Scanner logic out of AroundBlockScan introduce history
AroundBlockScan started as a utility class that was meant to be used as a DSL for scanning and making new blocks. As logic got added to this class it became hard to reason about what exactly is being mutated when. I pulled the scanning logic out into it's own class which gives us a clean separation of concerns. This allowed me to remove a lot of accessors that aren't core to the logic provided by AroundBlockScan.
In addition to this refactor the ScanHistory class can snapshot a scan. This allows us to be more aggressive with scans in the future as we can now snapshot and rollback if it didn't turn out the way we wanted.
The change comes with a minor perf impact:
before: 5.092678 0.104299 5.196977 ( 5.226494)
after: 5.128536 0.099871 5.228407 ( 5.249542)
This represents a 0.996x change in speed (where 1x would be no change and 2x would be twice as fast). This is a 0.38% decrease in performance which is negligible. It's likely coming from the extra blocks being created while scanning. This is negligible and if the history feature works well we might be able to make better block decisions which is means fewer calls to ripper which is the biggest bottleneck.
While this doesn't fix https://github.com/ruby/syntax_suggest/issues/187 it's a good intermediate step that will hopefully make working on that issue easier.
https://github.com/ruby/syntax_suggest/commit/ad8487d8aa
Diffstat (limited to 'lib/syntax_suggest/capture_code_context.rb')
| -rw-r--r-- | lib/syntax_suggest/capture_code_context.rb | 6 |
1 files changed, 5 insertions, 1 deletions
diff --git a/lib/syntax_suggest/capture_code_context.rb b/lib/syntax_suggest/capture_code_context.rb index a618b2ec68..71f5b271b5 100644 --- a/lib/syntax_suggest/capture_code_context.rb +++ b/lib/syntax_suggest/capture_code_context.rb @@ -55,6 +55,10 @@ module SyntaxSuggest capture_falling_indent(block) end + sorted_lines + end + + def sorted_lines @lines_to_output.select!(&:not_empty?) @lines_to_output.uniq! @lines_to_output.sort! @@ -116,7 +120,7 @@ module SyntaxSuggest return unless block.visible_lines.count == 1 around_lines = AroundBlockScan.new(code_lines: @code_lines, block: block) - .capture_neighbor_context + .capture_before_after_kws around_lines -= block.lines |