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

山本です。

>module_functionは、インスタンスメソッドをprivateにして、public
>なクラスメソッドにコピーします。なので、たしかにそこでprivateを
>指定しておくのは意味がありません。

>ついでに、forでループさせなくてもmodule_functionは複数同時に指
>定できます。
>
>    module_function(*instance_methods(false))
>    module_function(*private_instance_methods(false))

なるほど。private_class_method を使うとモジュール関数側も private
にできるようなので、まとめるとこんな感じでしょうか。

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 10:16:14 -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,26 +196,21 @@ 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
   #
-  private
   def symbol_to_option(symbol)
     if symbol.is_a? Integer
       symbol = CONSTANT_TO_SYMBOL[symbol]
@@ -236,7 +223,13 @@ module Kconv
       return nil
     end
   end
-  module_function :symbol_to_option
+
+  #
+  # Make them module functions
+  #
+  module_function(*instance_methods(false))
+  private_class_method :symbol_to_option
+
 end
 
 class String