Hi,

At Tue, 10 May 2005 10:04:30 +0900,
me2faster / excite.com wrote in [ruby-talk:141925]:
> Yes, the patch was not applied correctly, since I had both the
> following lines, rather than just the second one:
> 
> ....
> -    (tmp__protect_tmp = rb_node_newnode(NODE_ALLOCA,			\
> +    (tmp__protect_tmp = NEW_NODE(NODE_ALLOCA,			\
> ....
> 
> However, the original errors persist:

Even if you deleted the first `rb_node_newnode' line and `+' sign at
the beginning of the second one?

> And at least the first error (and I suspect the subsequent errors, too)
> related to the following pre-processed code appears to be valid:
> 
>                  argv = (tmp__protect_tmp =
> rb_node_newnode(NODE_ALLOCA, (VALUE*
> )ruby_xmalloc(sizeof(VALUE)*(argc)),tmp__protect_tmp,argc),
> (void*)tmp__protect_
> tmp->u1 . node);
> 
>  ^
> "/home/ruby-1.8.2/temp/eval.c", line 5164: error(252): argument of
>           type "VALUE *" is incompatible with parameter of type "VALUE"

The patch would make the latter three arguments of rb_node_newnode()
cast to VALUE.

-- 
Nobu Nakada