[ruby-dev:27664] Re: refactored shellwords.rb has bug?

なかだです。


At Thu, 10 Nov 2005 10:37:01 +0900,
KIMURA Koichi wrote in [ruby-dev:27663]:
> * lib/shellwords.rb: refactored.  [ruby-core:06581]
> 
> の変更により、CVS HEADがmswin32でビルドできなくなっているようです。

すいません、手元ではできていたもので。

> 調べてみたところ、shellwordsに空白だけからなるlineが渡されてくるようで、
> それにより
> 
>     raise ArgumentError, "Unmatched double quote: #{line}" if line[last]
> 
> の条件に引っかかり(line[last]にはスペースが入っている)、

修正方法はいくつか考えられますが、こんなところでどうでしょうか。
line.scanで$~をセットしなくてもいいようにしました。


Index: lib/shellwords.rb
===================================================================
RCS file: /cvs/ruby/src/ruby/lib/shellwords.rb,v
retrieving revision 1.9
diff -U2 -p -u -r1.9 shellwords.rb
--- lib/shellwords.rb	8 Nov 2005 23:41:40 -0000	1.9
+++ lib/shellwords.rb	10 Nov 2005 02:42:57 -0000
@@ -27,14 +27,11 @@ module Shellwords
   #
   def shellwords(line)
-    line = String.new(line) rescue
-      raise(ArgumentError, "Argument must be a string")
     words = []
     field = ''
-    last = 0
-    sep = nil
-    line.scan(/\G\s*(?:([^\s\\\'\"]+)|'([^\']*)'|"((?:[^\"\\]|\\.)*)"|(\\.?))(\s+|\z)?/m) do
-      last = $~.end(0)
-      sep = $~.begin(5)
-      field << ($1 || $2 || ($3 || $4).gsub(/\\(?=.)/, ''))
+    word = sq = dq = esc = garbage = sep = nil
+    line.scan(/\G\s*(?>([^\s\\\'\"]+)|'([^\']*)'|"((?:[^\"\\]|\\.)*)"|(\\.?)|(\S))(\s|\z)?/m) do
+      |word, sq, dq, esc, garbage, sep|
+      raise ArgumentError, "Unmatched double quote: #{line.inspect}" if garbage
+      field << (word || sq || (dq || esc).gsub(/\\(?=.)/, ''))
       if sep
         words << field
@@ -42,5 +39,4 @@ module Shellwords
       end
     end
-    raise ArgumentError, "Unmatched double quote: #{line}" if line[last]
     words
   end


-- 
--- 僕の前にBugはない。
--- 僕の後ろにBugはできる。
    中田 伸悦