[ruby-dev:22820] Re: File.fnmatch が "*?" のパターンを正しく処理しない？

山本です。

バグの修正と同時に大量にスタイルを修正するのは、
バグの所在がわかりにくくなって、よくない気がしてきました。

まずバグを修正したいと思います。そして改めて

・長すぎる行を分けたい
・空行を調節したい (特にglob_helperの後半）
・コメントをわかりやすくしたい
・変数名を整理したい (need_plain が特に変）
・escapeなどの変数をconstにしたい

などのスタイル修正をしたいと思います。

下のコードでコミットしてもいいでしょうか？


Index: dir.c
===================================================================
RCS file: /ruby/ruby/dir.c,v
retrieving revision 1.100
diff -u -w -b -r1.100 dir.c
--- dir.c	29 Jan 2004 11:59:55 -0000	1.100
+++ dir.c	7 Feb 2004 04:55:52 -0000
@@ -267,7 +267,7 @@
 
 	    test = escape && c == '\\' ? pat+1 : pat;
 	    while (*s) {
-		if ((c == '[' || Compare(s, test) == 0) &&
+		if ((c == '?' || c == '[' || Compare(s, test) == 0) &&
 		    !fnmatch(pat, s, flags | FNM_DOTMATCH))
 		    return 0;
 		else if (ISDIRSEP(*s))
@@ -288,7 +288,7 @@
 	  case '\\':
 	    if (escape && pat[1]
 #if defined DOSISH
-		&& strchr("*?[\\", pat[1])
+		&& strchr("*?[]\\", pat[1])
 #endif
 		) {
 		c = *++pat;