Hi,

At Sat, 9 Jul 2011 12:24:04 +0900,
Michael Edgar wrote in [ruby-core:37908]:
> The funny thing is, the parser already *is* doing almost all
> of the work! It's just not looking at the local variable
> tables when it comes time to generate the Ripper event. The
> patch I've attached does do so - it's a small change for a
> huge benefit for Ripper users.

I don't think 'self', 'nil', 'true', and so on are vcall.  Of
course they are neither really variables, but a kind of it
syntactically.  Reviewing from this point, your patch seems
wrong about use of get_id(), and I suspect it might need to
split the "variable" rule.

-- 
Nobu Nakada