ruby-list

  野首と申します。

  Namazu Project の方で regex.c を使わせていただいているのですが、これ
に対して神戸さんが警告を減らすパッチを作成されました。利用していた 
regex.c が古かったので、最新の安定版に適用できる形に直してこちらにも一
応お送りします。

  Ruby 1.6.6 の regex.c に対するパッチで、それぞれ以下のような変更を施
しています。

* patch-ruby-regex-warn-fix-1
  cast, 変数の初期化、||, && 演算子近辺への () の付加など

* patch-ruby-regex-warn-fix-2
  実際には利用していない関数や enum symbol のコメントアウト

  後者はなんらかの意図があって残してありそうなので、そのまま適用できそ
うな部分と分けました。これらのパッチの採用/不採用に関してはおまかせし
ますので、よろしくお願いします。
-- 
野首 貴嗣
E-mail: knok@daionet.gr.jp
	knok@namazu.org / knok@debian.org

Attachments (2)

patch-ruby-regex-warn-fix-1 (2.78 KB, text/x-diff)

diff -u regex-orig/regex.c regex-new/regex.c
--- regex-orig/regex.c	Fri Jan 18 15:55:21 2002
+++ regex-new/regex.c	Fri Jan 18 15:43:05 2002
@@ -376,7 +376,7 @@
     wordbeg,	 /* Succeeds if at word beginning.  */
     wordend,	 /* Succeeds if at word end.  */
     wordbound,   /* Succeeds if at a word boundary.  */
-    notwordbound,/* Succeeds if not at a word boundary.  */
+    notwordbound /* Succeeds if not at a word boundary.  */
   };
 
 
@@ -461,7 +461,7 @@
       int n = mbclen(c) - 1;						\
       c &= (1<<(BYTEWIDTH-2-n)) - 1;					\
       while (n--) {							\
-	c = c << 6 | *p++ & ((1<<6)-1);					\
+	c = c << 6 | (*p++ & ((1<<6)-1));				\
       }									\
     }									\
     else {								\
@@ -502,23 +502,28 @@
 {
   if (current_mbctype == MBCTYPE_UTF8) {
     if (c < 0x80)
-      printf("%c", c);
+      printf("%c", (int)c);
     else if (c <= 0x7ff)
-      printf("%c%c", utf8_firstbyte(c), c&0x3f);
+      printf("%c%c", (int)utf8_firstbyte(c), (int)(c & 0x3f));
     else if (c <= 0xffff)
-      printf("%c%c%c", utf8_firstbyte(c), (c>>6)&0x3f, c&0x3f);
+      printf("%c%c%c", (int)utf8_firstbyte(c), (int)((c >> 6) & 0x3f),
+	     (int)(c & 0x3f));
     else if (c <= 0x1fffff) 
-      printf("%c%c%c%c", utf8_firstbyte(c), (c>>12)&0x3f, (c>>6)&0x3f, c&0x3f);
+      printf("%c%c%c%c", (int)utf8_firstbyte(c), (int)((c >> 12) & 0x3f),
+	     (int)((c >> 6) & 0x3f), (int)(c & 0x3f));
     else if (c <= 0x3ffffff)
-      printf("%c%c%c%c%c", utf8_firstbyte(c), (c>>18)&0x3f, (c>>12)&0x3f, (c>>6)&0x3f, c&0x3f);
+      printf("%c%c%c%c%c", (int)utf8_firstbyte(c), (int)((c >> 18) & 0x3f),
+	     (int)((c >> 12) & 0x3f), (int)((c >> 6) & 0x3f), (int)(c & 0x3f));
     else if (c <= 0x7fffffff)
-      printf("%c%c%c%c%c%c", utf8_firstbyte(c), (c>>24)&0x3f, (c>>18)&0x3f, (c>>12)&0x3f, (c>>6)&0x3f, c&0x3f);
+      printf("%c%c%c%c%c%c", (int)utf8_firstbyte(c), (int)((c >> 24) & 0x3f),
+	     (int)((c >> 18) & 0x3f), (int)((c >> 12) & 0x3f),
+	     (int)((c >> 6) & 0x3f), (int)(c & 0x3f));
   }
   else if (c < 0xff) {
-    printf("\\%o", c);
+    printf("\\%o", (int)c);
   }
   else {
-    printf("%c%c", c>>BYTEWIDTH, c&0xff);
+    printf("%c%c", (int)(c >> BYTEWIDTH), (int)(c &0xff));
   }
 }
 
@@ -1184,7 +1189,7 @@
   register const char *p = pattern;
   const char *nextp;
   const char *pend = pattern + size;
-  register unsigned int c, c1;
+  register unsigned int c, c1 = 0;
   const char *p0;
   int numlen;
 #define ERROR_MSG_MAX_SIZE 200
@@ -1484,8 +1489,8 @@
 	  case 'W':
 	    for (c = 0; c < (1 << BYTEWIDTH); c++) {
 	      if (SYNTAX(c) != Sword &&
-		  (current_mbctype && !re_mbctab[c] ||
-		  !current_mbctype && SYNTAX(c) != Sword2))
+		  ((current_mbctype && !re_mbctab[c]) ||
+		  (!current_mbctype && SYNTAX(c) != Sword2)))
 		SET_LIST_BIT(c);
 	    }
 	    had_char_class = 1;

patch-ruby-regex-warn-fix-2 (3.85 KB, text/x-diff)

diff -u regex-fix1/regex-ruby.c regex-new/regex-ruby.c
--- regex-fix1/regex-ruby.c	Fri Jan 18 15:58:24 2002
+++ regex-new/regex-ruby.c	Fri Jan 18 16:01:36 2002
@@ -169,7 +169,9 @@
 static void insert_jump _((int, char*, char*, char*));
 static void store_jump_n _((char*, int, char*, unsigned));
 static void insert_jump_n _((int, char*, char*, char*, unsigned));
+#if 0
 static void insert_op _((int, char*, char*));
+#endif
 static void insert_op_2 _((int, char*, char*, int, int));
 static int memcmp_translate _((unsigned char*, unsigned char*, int));
 
@@ -370,7 +372,9 @@
     duplicate,   /* Match a duplicate of something remembered.
 		    Followed by one byte containing the index of the memory 
                     register.  */
+#if 0
     fail,        /* always fails. */
+#endif
     wordchar,    /* Matches any word-constituent character.  */
     notwordchar, /* Matches any char that is not a word-constituent.  */
     wordbeg,	 /* Succeeds if at word beginning.  */
@@ -724,6 +728,7 @@
   return 0;
 }
 
+#if 0
 static void
 print_partial_compiled_pattern(start, end)
     unsigned char *start;
@@ -978,6 +983,7 @@
 
   print_partial_compiled_pattern(buffer, buffer + bufp->used);
 }
+#endif
 
 static char*
 calculate_must_string(start, end)
@@ -1251,7 +1257,9 @@
   int *stackb = stacka;
   int *stackp = stackb;
   int *stacke = stackb + 40;
+#if 0
   int *stackt;
+#endif
 
   /* Counts ('s as they are encountered.  Remembered for the matching ),
      where it becomes the register number to put in the stop_memory
@@ -2567,6 +2575,7 @@
 
    If you call this function, you must zero out pending_exact.  */
 
+#if 0
 static void
 insert_op(op, there, current_end)
      int op;
@@ -2580,6 +2589,7 @@
 
   there[0] = (char)op;
 }
+#endif
 
 
 /* Open up space at location THERE, and insert operation OP followed by
diff -u regex-fix1/regex.c regex-new/regex.c
--- regex-fix1/regex.c	Fri Jan 18 15:58:24 2002
+++ regex-new/regex.c	Fri Jan 18 16:03:56 2002
@@ -169,7 +169,9 @@
 static void insert_jump _((int, char*, char*, char*));
 static void store_jump_n _((char*, int, char*, unsigned));
 static void insert_jump_n _((int, char*, char*, char*, unsigned));
+#if 0
 static void insert_op _((int, char*, char*));
+#endif
 static void insert_op_2 _((int, char*, char*, int, int));
 static int memcmp_translate _((unsigned char*, unsigned char*, int));
 
@@ -370,7 +372,9 @@
     duplicate,   /* Match a duplicate of something remembered.
 		    Followed by one byte containing the index of the memory 
                     register.  */
+#if 0
     fail,        /* always fails. */
+#endif
     wordchar,    /* Matches any word-constituent character.  */
     notwordchar, /* Matches any char that is not a word-constituent.  */
     wordbeg,	 /* Succeeds if at word beginning.  */
@@ -496,6 +500,7 @@
 #endif
 }
 
+#if 0
 static void
 print_mbc(c)
      unsigned int c;
@@ -526,6 +531,7 @@
     printf("%c%c", (int)(c >> BYTEWIDTH), (int)(c &0xff));
   }
 }
+#endif
 
 /* If the buffer isn't allocated when it comes in, use this.  */
 #define INIT_BUF_SIZE  28
@@ -724,6 +730,7 @@
   return 0;
 }
 
+#if 0
 static void
 print_partial_compiled_pattern(start, end)
     unsigned char *start;
@@ -978,6 +985,7 @@
 
   print_partial_compiled_pattern(buffer, buffer + bufp->used);
 }
+#endif
 
 static char*
 calculate_must_string(start, end)
@@ -1251,7 +1259,9 @@
   int *stackb = stacka;
   int *stackp = stackb;
   int *stacke = stackb + 40;
+#if 0
   int *stackt;
+#endif
 
   /* Counts ('s as they are encountered.  Remembered for the matching ),
      where it becomes the register number to put in the stop_memory
@@ -2567,6 +2577,7 @@
 
    If you call this function, you must zero out pending_exact.  */
 
+#if 0
 static void
 insert_op(op, there, current_end)
      int op;
@@ -2580,6 +2591,7 @@
 
   there[0] = (char)op;
 }
+#endif
 
 
 /* Open up space at location THERE, and insert operation OP followed by

Thread

Prev Next

In This Thread

Prev Next

[#33152] ruby が core を吐く — 堀川 久 <vzw00011@...>

[#33156] ライブラリ中の DATA — Kazuhiro NISHIYAMA <zn@...>

[#33169] a cool Ruby script. — Yuji Asakawa <namu@...>

[#33171] Fw: ANNOUNCE: Inline::Ruby 0.01-pre2 — Tatsuhiko Miyagawa <miyagawa@...>

[#33172] [ANN] Ruby/zlib 0.5.0-pre1 — UENO Katsuhiro <unnie@...>

[#33175] [ANN] irbsh-0.5.2 — rubikitch <rubikitch@...>

[#33176] ruby から vi の呼び出し — masami kumazawa <kumazawa@...>

[#33182] RegexpError — Masaki Suketa <masaki.suketa@...>

[#33183] [ANN] TMail 0.10 — Minero Aoki <aamine@...>

[#33185] rd-mode: rd-intelligent-newline — "Shirai,Kaoru" <shirai@...1jp.com>

[#33193] multipart 切り出しライブラリ — encore ann <i@...>

[#33204] [ANN] HTML/FillInForm 0.21 — IKEBE Tomohiro <ikebe@...>

[#33208] Method passing block — sinara@...

[#33209] [ANN] setup.rb 3.1.0 — Minero Aoki <aamine@...>

[#33215] [ANN] RubyCocoa 0.2.1 — FUJIMOTO Hisakuni <hisa@...>

[#33217] setup.rb 3.1.0 — Minero Aoki <aamine@...>

[#33228] [ANN] RubyCocoa 0.2.2 — FUJIMOTO Hisakuni <hisa@...>

[#33229] ruby のインストールがうまくいかない — satoshi SAKAI <gau_sato@...9.dion.ne.jp>

[#33234] DB のカラム名をもった各行のオブジェクトへ向けて — encore ann <i@...>

[#33235] " 差分ベースモジュール " in Ruby — Taku Nakajima <tnakajima@...>

[#33263] RDtool インストールでの疑問 — satoshi SAKAI <gau_sato@...9.dion.ne.jp>

[#33284] [ANN] irbsh-0.5.3 — rubikitch <rubikitch@...>

[#33288] HTML から BODY を抜き出す — m_seki@...

[#33294] [PATCH] rdtool on ruby 1.4(.6) — Minero Aoki <aamine@...>

[#33300] 日付順のソートについて — "Tatsuaki Nagai" <Tatsuaki_Nagai@...>

[#33305] Re: 日付順のソートについて — "Tatsuaki Nagai" <Tatsuaki_Nagai@...>

[#33309] Re: 日付順のソートについて — "Tatsuaki Nagai" <Tatsuaki_Nagai@...>

[#33311] Re: 日付順のソートについて — "Tatsuaki Nagai" <Tatsuaki_Nagai@...>

[#33313] Re: 日付順のソートについて — "Tatsuaki Nagai" <Tatsuaki_Nagai@...>

[#33318] ［質問］正規表現について — keiichi matsunaga <ma2@...>

[#33326] [ 質問 ] クラスメソッドのモジュールによる実現方法 — Hiroyuki Komatsu <komatsu@...>

[#33335] [ANN] rdtool easy install package — Minero Aoki <aamine@...>

[#33342] waitpid2 が返す終了ステータス — みかみまさと <kofu@...>

[#33346] IPSocket.getaddress — "Sadayuki Sakamoto" <sakacom@...>

[#33355] setup.rb for static linkage — HGF01572@...

[#33356] about pty and expect on 1.7.x — YANAGAWA Kazuhisa <kjana@...4lab.to>

[#33358] $KCODE の影響範囲 — Masatoshi SEKI <m_seki@...>

[#33367] [ANN] rb2exe-preview1 — Yuya Kato <yuya@4th.to>

[#33381] [ANN] WString 0.0.8 — takuma ozawa <metal@...>

[#33392] Re: [ANN] rb2exe-preview1 — Nagai T <idonagai@...>

[#33397] Re: [ANN] rb2exe-preview1 — Nagai T <idonagai@...>

[#33398] ruby-mode のインデント不具合 (} + の次の行) — MALTA Koji <malta@...>

[#33399] Re: [ANN] rb2exe-preview1 — Nagai T <idonagai@...>

[#33412] Ruby/Becky! 0.9.7.1 — "T.Shimomura" <redbugml@...>

[#33413] lack of the parameter type of rb_gc_mark() — Takaaki Tateishi <ttate@...>

[#33415] add-log function for ruby-mode.el — TAKAHASHI Kaoru <kaoru@...>

[#33416] No such file to load -- dbm — satoshi SAKAI <gau_sato@...9.dion.ne.jp>

[#33427] CGI でキャッシュ — Fujikawa Norihide <ruby@...>

[#33431] [ANN]rdindex-1.1.7 — rubikitch <rubikitch@...>

[#33443] libpgsql-ruby-0.6.5 で Linux+PostgreSQL+Ruby について — "井上 浩一" <kyoui32@...>

[#33459] warning silence patch for regex.c — NOKUBI Takatsugu <knok@...>

[#33462] Re: Mysql/Ruby のコンパイル — "井上 浩一" <kyoui32@...>

[#33469] Ruby/Becky! 0.9.7.2 — "T.Shimomura" <redbugml@...>

[#33472] "The Ruby Way" — sinara@...

[#33477] about nest class — sheepman <sheepman@...>

[#33491] ファイル名を小文字に — 大道 裕 <daidou@...2.so-net.ne.jp>

[#33494] Re: ファイル名を小文字に — "Tatsuaki Nagai" <Tatsuaki_Nagai@...>

[#33515] Media Watch (月刊 ASCII 2 月号) — matz@... (Yukihiro Matsumoto)

[#33517] [BUG] Can't compile ruby on Solaris2.6 — IWAMURO Motonori <iwa@...>

[#33519] Re: libpgsql-ruby-0.6.5 で Linux+PostgreSQL+Ruby について — "井上 浩一" <kyoui32@...>

[#33520] IP に関するライブラリ cygwin & linux — "井上 浩一" <kyoui32@...>

[#33535] broken symbolic link and File::mtime — Go Nakagawa <nakagawa@...>

[#33538] Re: [rubyist:1056] Re: Re: 文字列→整数変換 (Re: 次期リリースに向けて今後) — maili31s@... (SugHimsi == SUGIHARA Hiroshi)

[#33547] racc fails on alpha-freebsd — Minero Aoki <aamine@...>

[#33556] Cgi::session in eRuby — kyosuke kiyooka <k@7ruby.com>

[#33570] NHK スペシャル「変革の世紀」 — rubiben <ozaki@...>

[#33605] ある文字列以外の部分にマッチさせる方法 — "Tatsuaki Nagai" <Tatsuaki_Nagai@...>

[#33612] Re: ある文字列以外の部分にマッチさせる方法 — "Tatsuaki Nagai" <Tatsuaki_Nagai@...>

[#33613] Re: ある文字列以外の部分にマッチさせる方法 — "Tatsuaki Nagai" <Tatsuaki_Nagai@...>

[#33620] Re: ある文字列以外の部分にマッチさせる方法 — "Tatsuaki Nagai" <Tatsuaki_Nagai@...>

[#33626] Re: ある文字列以外の部分にマッチさせる方法 — "Tatsuaki Nagai" <Tatsuaki_Nagai@...>

[#33627] Re: ある文字列以外の部分にマッチさせる方法 — "Tatsuaki Nagai" <Tatsuaki_Nagai@...>

[#33642] Re: ある文字列以外の部分にマッチさせる方法 — "Tatsuaki Nagai" <Tatsuaki_Nagai@...>

[#33652] make tmail loading faster — Minero Aoki <aamine@...>

[#33653] Re: ある文字列以外の部分にマッチさせる方法 — "Tatsuaki Nagai" <Tatsuaki_Nagai@...>

[#33655] Re: ある文字列以外の部分にマッチさせる方法 — "Tatsuaki Nagai" <Tatsuaki_Nagai@...>

[#33656] Re: ある文字列以外の部分にマッチさせる方法 — "Tatsuaki Nagai" <Tatsuaki_Nagai@...>

[#33659] [ANN]MySQL/Ruby Windows package — "TAMURA.KENICHI" <sgs02516@...>

[#33664] rd2 コマンド（ RDtool ）が動かない。 — Seiji Yoneda <yoneda@...>

[#33689] Windows をリブートするスクリプトはありますか？。 — Take_tk <ggb03124@...>

[#33152] ruby が core を吐く — 堀川久 <vzw00011@...>

[#33443] libpgsql-ruby-0.6.5 で Linux+PostgreSQL+Ruby について — "井上浩一" <kyoui32@...>

[#33462] Re: Mysql/Ruby のコンパイル — "井上浩一" <kyoui32@...>

[#33491] ファイル名を小文字に — 大道裕 <daidou@...2.so-net.ne.jp>

[#33519] Re: libpgsql-ruby-0.6.5 で Linux+PostgreSQL+Ruby について — "井上浩一" <kyoui32@...>

[#33520] IP に関するライブラリ cygwin & linux — "井上浩一" <kyoui32@...>

[#33695] 自作のクラスのテストをしていますが、うまく動作しません。 — 岩崎弘孝 <IH000667@...>

[#33698] Re: 自作のクラスのテストをしていますが、うまく動作しません。 — 岩崎弘孝 <IH000667@...>