diff options
author | aycabta <aycabta@gmail.com> | 2019-12-13 03:57:32 +0900 |
---|---|---|
committer | aycabta <aycabta@gmail.com> | 2019-12-13 08:54:22 +0900 |
commit | c2dfc6d869979124a46fb0c5404956891c27575f (patch) | |
tree | 45aa67fb11df82804dcc02d0cb2eb539696d01ae | |
parent | b8d6c883b3f49e6339da4fa5111dbdbe7d3c6df5 (diff) | |
download | ruby-c2dfc6d869979124a46fb0c5404956891c27575f.tar.gz |
Show a menu before a document
IRB should show a menu first if a completed list has plural items. But just
shows document without menu if a completed list with plural items includes a
perfect matched item. The behavior is a bug. This commit fixes it.
-rw-r--r-- | lib/reline/line_editor.rb | 20 | ||||
-rw-r--r-- | test/reline/test_key_actor_emacs.rb | 66 |
2 files changed, 81 insertions, 5 deletions
diff --git a/lib/reline/line_editor.rb b/lib/reline/line_editor.rb index 5d23c82ded..20d966f6dc 100644 --- a/lib/reline/line_editor.rb +++ b/lib/reline/line_editor.rb @@ -43,6 +43,7 @@ class Reline::LineEditor COMPLETION = :completion MENU = :menu JOURNEY = :journey + MENU_WITH_PERFECT_MATCH = :menu_with_perfect_match PERFECT_MATCH = :perfect_match end @@ -599,16 +600,24 @@ class Reline::LineEditor when CompletionState::PERFECT_MATCH @dig_perfect_match_proc&.(@perfect_matched) end - is_menu = (@completion_state == CompletionState::MENU) + is_menu = (@completion_state == CompletionState::MENU or @completion_state == CompletionState::MENU_WITH_PERFECT_MATCH) result = complete_internal_proc(list, is_menu) + if @completion_state == CompletionState::MENU_WITH_PERFECT_MATCH + @completion_state = CompletionState::PERFECT_MATCH + end return if result.nil? target, preposing, completed, postposing = result return if completed.nil? - if target <= completed and (@completion_state == CompletionState::COMPLETION or @completion_state == CompletionState::PERFECT_MATCH) - @completion_state = CompletionState::MENU + if target <= completed and (@completion_state == CompletionState::COMPLETION) if list.include?(completed) - @completion_state = CompletionState::PERFECT_MATCH + if list.one? + @completion_state = CompletionState::PERFECT_MATCH + else + @completion_state = CompletionState::MENU_WITH_PERFECT_MATCH + end @perfect_matched = completed + else + @completion_state = CompletionState::MENU end if target < completed @line = preposing + completed + postposing @@ -622,7 +631,8 @@ class Reline::LineEditor private def move_completed_list(list, direction) case @completion_state - when CompletionState::NORMAL, CompletionState::COMPLETION, CompletionState::MENU + when CompletionState::NORMAL, CompletionState::COMPLETION, + CompletionState::MENU, CompletionState::MENU_WITH_PERFECT_MATCH @completion_state = CompletionState::JOURNEY result = retrieve_completion_block return if result.nil? diff --git a/test/reline/test_key_actor_emacs.rb b/test/reline/test_key_actor_emacs.rb index 4e3184bd3f..7ffa28ec87 100644 --- a/test/reline/test_key_actor_emacs.rb +++ b/test/reline/test_key_actor_emacs.rb @@ -1287,6 +1287,72 @@ class Reline::KeyActor::Emacs::Test < Reline::TestCase assert_line('foo_ba') end + def test_completion_with_perfect_match + @line_editor.completion_proc = proc { |word| + %w{ + foo + foo_bar + }.map { |i| + i.encode(@encoding) + } + } + matched = nil + @line_editor.dig_perfect_match_proc = proc { |m| + matched = m + } + input_keys('fo') + assert_byte_pointer_size('fo') + assert_cursor(2) + assert_cursor_max(2) + assert_line('fo') + assert_equal(Reline::LineEditor::CompletionState::NORMAL, @line_editor.instance_variable_get(:@completion_state)) + assert_equal(nil, matched) + input_keys("\C-i", false) + assert_byte_pointer_size('foo') + assert_cursor(3) + assert_cursor_max(3) + assert_line('foo') + assert_equal(Reline::LineEditor::CompletionState::MENU_WITH_PERFECT_MATCH, @line_editor.instance_variable_get(:@completion_state)) + assert_equal(nil, matched) + input_keys("\C-i", false) + assert_byte_pointer_size('foo') + assert_cursor(3) + assert_cursor_max(3) + assert_line('foo') + assert_equal(Reline::LineEditor::CompletionState::PERFECT_MATCH, @line_editor.instance_variable_get(:@completion_state)) + assert_equal(nil, matched) + input_keys("\C-i", false) + assert_byte_pointer_size('foo') + assert_cursor(3) + assert_cursor_max(3) + assert_line('foo') + assert_equal(Reline::LineEditor::CompletionState::PERFECT_MATCH, @line_editor.instance_variable_get(:@completion_state)) + assert_equal('foo', matched) + matched = nil + input_keys('_') + input_keys("\C-i", false) + assert_byte_pointer_size('foo_bar') + assert_cursor(7) + assert_cursor_max(7) + assert_line('foo_bar') + assert_equal(Reline::LineEditor::CompletionState::MENU_WITH_PERFECT_MATCH, @line_editor.instance_variable_get(:@completion_state)) + assert_equal(nil, matched) + input_keys("\C-i", false) + assert_byte_pointer_size('foo_bar') + assert_cursor(7) + assert_cursor_max(7) + assert_line('foo_bar') + assert_equal(Reline::LineEditor::CompletionState::PERFECT_MATCH, @line_editor.instance_variable_get(:@completion_state)) + assert_equal(nil, matched) + input_keys("\C-i", false) + assert_byte_pointer_size('foo_bar') + assert_cursor(7) + assert_cursor_max(7) + assert_line('foo_bar') + assert_equal(Reline::LineEditor::CompletionState::PERFECT_MATCH, @line_editor.instance_variable_get(:@completion_state)) + assert_equal('foo_bar', matched) + end + def test_completion_with_completion_ignore_case @line_editor.completion_proc = proc { |word| %w{ |