Ryan Davis wrote:
>
> On Sep 29, 2009, at 01:03 , Mark Moseley wrote:
>
>> Ryan Davis wrote:
>>>> /**
>>>>  @c setting
>>>>  @e trace
>>>>  @j trace Ѥ̿ᡣ
>>>> */
>>>> DEFINE_INSN
>>>> trace
>>>> (rb_num_t nf)
>>>> ()
>>>> ()
>>>> {
>>>>    rb_event_flag_t flag = (rb_event_flag_t)nf;
>>>>
>>>>    EXEC_EVENT_HOOK(th, flag, GET_SELF(), 0, 0 /* TODO: id, klass */);
>>>> }
>>>
>>> Am I missing something? Does this TODO imply that the event hook 
>>> mechanism isn't supposed to work yet?
>>>
>>>
>>>
>>>
>>
>> The hook works fine, it just doesn't pass in the id and klass 
>> variables. [...]
>
> No... "works fine" for me means "compatible" and this definitely 
> isn't. My event_hook gem has had to go through a lot of convolutions 
> just to compile and now it turns out that it is going to need even 
> more to work as it did before.
>
>
>
>
Well, you asked about "supposed to work". I think that the answer to 
that is yes. But no, it's not backwards-compatible at all with 1.8. I 
would also say that the more that I dig into the Ruby VM the more I like 
it. To each his own, though, as you pointed out.