[ruby-dev:27430] Re: cannot build ruby with byacc (1.8)

なかだです。

At Mon, 17 Oct 2005 16:04:16 +0900,
U.Nakamura wrote in [ruby-dev:27428]:
> 手元にあるbyacc使用でコンパイルしたrubyの一番最近のものが
> 2005-10-05なので、10/08のparse.yの変更か、または10/13のparse.y
> の変更か、どちらか(あるいは両方)が問題なのだと思われます。

両方ですね。というか10/13のほうは10/08のbugfixなので。

> 1.8までのmswin32はbyaccを使用するようになっているので、修正す
> るのか、bison必須にするのか、方針を教えてください。

byaccはparser stackを解放せずに使いまわすようですねぇ。


Index: parse.y
===================================================================
RCS file: /cvs/ruby/src/ruby/parse.y,v
retrieving revision 1.307.2.17
diff -U2 -p -r1.307.2.17 parse.y
--- parse.y	13 Oct 2005 14:26:00 -0000	1.307.2.17
+++ parse.y	17 Oct 2005 09:54:38 -0000
@@ -2546,5 +2546,7 @@ int ruby__end__seen;
 
 static VALUE ruby_debug_lines;
+#ifdef YYMALLOC
 static NODE *parser_heap;
+#endif
 
 static NODE*
@@ -2586,5 +2588,4 @@ yycompile(f, line)
     ruby_current_node = 0;
     ruby_sourcefile = rb_source_filename(f);
-    parser_heap = 0;
     n = yyparse();
     ruby_debug_lines = 0;
@@ -2598,5 +2599,4 @@ yycompile(f, line)
     in_def = 0;
     cur_mid = 0;
-    parser_heap = 0;
 
     vp = ruby_dyna_vars;
@@ -3013,5 +3013,5 @@ dispose_string(str)
     VALUE str;
 {
-    free(RSTRING(str)->ptr);
+    xfree(RSTRING(str)->ptr);
     rb_gc_force_recycle(str);
 }
@@ -5584,9 +5584,9 @@ local_pop()
 
     if (lvtbl->tbl) {
-	if (!lvtbl->nofree) free(lvtbl->tbl);
+	if (!lvtbl->nofree) xfree(lvtbl->tbl);
 	else lvtbl->tbl[0] = lvtbl->cnt;
     }
     ruby_dyna_vars = lvtbl->dyna_vars;
-    free(lvtbl);
+    xfree(lvtbl);
     lvtbl = local;
 }
@@ -5696,5 +5696,5 @@ top_local_setup()
 	    }
 	    if (ruby_scope->local_tbl && ruby_scope->local_vars[-1] == 0) {
-		free(ruby_scope->local_tbl);
+		xfree(ruby_scope->local_tbl);
 	    }
 	    ruby_scope->local_vars[-1] = 0;
@@ -5757,4 +5757,5 @@ void
 rb_gc_mark_parser()
 {
+    rb_gc_mark((VALUE)parser_heap);
     if (!ruby_in_compile) return;
 
@@ -5764,5 +5765,4 @@ rb_gc_mark_parser()
     rb_gc_mark(lex_input);
     rb_gc_mark((VALUE)lex_strterm);
-    rb_gc_mark((VALUE)parser_heap);
 }
 


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