Fix completion of IO::o

Author: drbrain

History.txt

@@ -10,7 +10,8 @@
     Ruby commit r30811 by Usaku Nakamura.
   * Remove unnecessary (and wrong) platform-dependent hacks.  Ruby commit
     r30829 by Usaku Nakamura.
-  * Completing via Array#[<tab> in `ri -i` no longer crashes.
+  * Completing via Array#[ in `ri -i` no longer crashes.  Ruby Bug #3167
+  * Completing IO::o in `ri -i` now returns results.  Ruby Bug #3167
 
 === 3.5.3 / 2010-02-06
 

lib/rdoc/ri/driver.rb

@@ -526,6 +526,8 @@ def complete name
     if name !~ /#|\./ then
       completions = klasses.grep(/^#{klass_name}[^:]*$/)
       completions.concat klasses.grep(/^#{name}[^:]*$/) if name =~ /::$/
+
+      completions << klass if classes.key? klass # to complete a method name
     elsif selector then
       completions << klass if classes.key? klass
     elsif classes.key? klass_name then
@@ -547,7 +549,7 @@ def complete name
       completions.push(*methods)
     end
 
-    completions.sort
+    completions.sort.uniq
   end
 
   ##

test/test_rdoc_ri_driver.rb

@@ -3,6 +3,7 @@
 require 'minitest/autorun'
 require 'tmpdir'
 require 'fileutils'
+require 'stringio'
 require 'rdoc/ri/driver'
 
 class TestRDocRIDriver < MiniTest::Unit::TestCase
@@ -252,9 +253,11 @@ def test_complete
     assert_equal %w[Foo         ], @driver.complete('F')
     assert_equal %w[    Foo::Bar], @driver.complete('Foo::B')
 
-    assert_equal %w[Foo#Bar],           @driver.complete('Foo#'),  'Foo#'
-    assert_equal %w[Foo#Bar  Foo::bar], @driver.complete('Foo.'),  'Foo.'
-    assert_equal %w[Foo::Bar Foo::bar], @driver.complete('Foo::'), 'Foo::'
+    assert_equal %w[Foo#Bar],           @driver.complete('Foo#'),   'Foo#'
+    assert_equal %w[Foo#Bar  Foo::bar], @driver.complete('Foo.'),   'Foo.'
+    assert_equal %w[Foo::Bar Foo::bar], @driver.complete('Foo::'),  'Foo::'
+
+    assert_equal %w[         Foo::bar], @driver.complete('Foo::b'), 'Foo::b'
   end
 
   def test_complete_ancestor