[ruby-dev:26808] test/nkf/test_kconv.rb

山本です。

HEAD で nkf のテストがエラーになっています。

E:\ruby-cvs\bcc32>testrb nkf
Loaded suite nkf
Started
EEEE.
Finished in 0.01 seconds.

  1) Error:
test_eucjp(TestKconv):
NoMethodError: undefined method `kconv' for Kconv:Module
    E:/ruby-cvs/bcc32/.ext/kconv.rb:244:in `kconv'
    E:/ruby-cvs/ruby/test/nkf/test_kconv.rb:31:in `test_eucjp'

  2) Error:
test_jis(TestKconv):
NoMethodError: undefined method `kconv' for Kconv:Module
    E:/ruby-cvs/bcc32/.ext/kconv.rb:244:in `kconv'
    E:/ruby-cvs/ruby/test/nkf/test_kconv.rb:66:in `test_jis'

  3) Error:
test_shiftjis(TestKconv):
NoMethodError: undefined method `kconv' for Kconv:Module
    E:/ruby-cvs/bcc32/.ext/kconv.rb:244:in `kconv'
    E:/ruby-cvs/ruby/test/nkf/test_kconv.rb:43:in `test_shiftjis'

  4) Error:
test_utf8(TestKconv):
NoMethodError: undefined method `kconv' for Kconv:Module
    E:/ruby-cvs/bcc32/.ext/kconv.rb:244:in `kconv'
    E:/ruby-cvs/ruby/test/nkf/test_kconv.rb:55:in `test_utf8'

5 tests, 25 assertions, 0 failures, 4 errors

調べてみたところ、

  http://www.ruby-lang.org/ja/man/index.cgi?cmd=view;name=Module#module_function

で理由を見つけました。モジュール関数を alias しても、新しくできた関数は
モジュール関数にならないそうです。

とりあえず下のようにして動くことを確認しました。

Index: kconv.rb
===================================================================
RCS file: /src/ruby/ext/nkf/lib/kconv.rb,v
retrieving revision 1.11
diff -u -w -b -p -r1.11 kconv.rb
--- kconv.rb	12 Aug 2005 23:19:58 -0000	1.11
+++ kconv.rb	17 Aug 2005 08:52:46 -0000
@@ -52,8 +52,11 @@ module Kconv
 		   )*\z/nx
   
   # SYMBOL_TO_OPTION is the table for Kconv#conv

+
   # Kconv#conv is intended to generic convertion method,

+
   # so this table specifies symbols which can be supported not only nkf...

+
   SYMBOL_TO_OPTION = {
     :iso2022jp	=> '-j',
     :jis	=> '-j',
@@ -118,9 +121,9 @@ module Kconv
     nkf_opt = '-x -m0 %s %s %s' % [to, from, opt]
     result = ::NKF::nkf( nkf_opt, str)
   end
-  module_function :conv
   alias :kconv :conv

 
+
   #
   # Encode to
   #
@@ -128,27 +131,22 @@ module Kconv
   def tojis(str)
     ::NKF::nkf('-j', str)
   end
-  module_function :tojis
 
   def toeuc(str)
     ::NKF::nkf('-e', str)
   end
-  module_function :toeuc
 
   def tosjis(str)
     ::NKF::nkf('-s', str)
   end
-  module_function :tosjis
 
   def toutf8(str)
     ::NKF::nkf('-w', str)
   end
-  module_function :toutf8
 
   def toutf16(str)
     ::NKF::nkf('-w16', str)
   end
-  module_function :toutf16
 
   alias :to_jis :tojis
   alias :to_euc :toeuc
@@ -166,17 +164,14 @@ module Kconv
   def guess(str)
     ::NKF::guess(str)
   end
-  module_function :guess
 
   def guess_old(str)
     ::NKF::guess1(str)
   end
-  module_function :guess_old
 
   def guess_as_symbol(str)
     CONSTANT_TO_SYMBOL[guess(str)]
   end
-  module_function :guess_as_symbol
 
   #
   # isEncoding
@@ -185,17 +180,14 @@ module Kconv
   def iseuc(str)
     RegexpEucjp.match( str )
   end
-  module_function :iseuc
   
   def issjis(str)
     RegexpShiftjis.match( str )
   end
-  module_function :issjis
 
   def isutf8(str)
     RegexpUtf8.match( str )
   end
-  module_function :isutf8
 
   #
   # encoding?
@@ -204,21 +196,17 @@ module Kconv
   def eucjp?(str)
     RegexpEucjp.match( str ) ? true : false
   end
-  module_function :eucjp?
 
   def shiftjis?(str)
     RegexpShiftjis.match( str ) ? true : false
   end
-  module_function :shiftjis?
+
   def utf8?(str)
     RegexpUtf8.match( str ) ? true : false
   end
-  module_function :utf8?
+
   alias :euc? :eucjp?
   alias :sjis? :shiftjis?
-  module_function :euc?
-  module_function :sjis?
-
 
   #
   # Private Methods
@@ -236,7 +224,14 @@ module Kconv
       return nil
     end
   end
-  module_function :symbol_to_option
+
+  #
+  # Make them module functions
+  #
+  for m in instance_methods(false) + private_instance_methods(false)
+    module_function m
+  end
+
 end
 
 class String

# ところで、module はよく知らないのですが、private なのに Kconv.symbole_to_option(:sjis)
# として呼び出せるのは正しい挙動なんでしょうか？private の意味がないような・・・