Btw, I think you've introduced a bug with one of your changes. Paste this into Emacs:

return if safe_assignment?(cond) && safe_assignment_allowed?

"?(" and the last "?" are highlighted with the font-lock-string face.


On 15 July 2013 16:53, Bozhidar Batsov <bozhidar@batsov.com> wrote:



On 13 July 2013 22:10, Dmitry Gutov <dgutov@yandex.ru> wrote:
On 13.07.2013 14:23, Bozhidar Batsov wrote:
Great! Btw, shouldn't we highlight the conversion methods from Kernel
(Float, Integer, etc) as built-ins?

Sure, it's now easier to do correctly. Done. :)

Thanks!
 


    But there's really nothing that would distinguish a Math::E
    reference from Foo::C. Or from GC, IO, DL, IRB, URI, XML, HTML, etc.


Yep, I'm aware we can't distinguish all cases without the use of a
proper parser, but I guess we should handle accordingly at least the
scenarios which are deterministic.

But we have to highlight these abbreviation-named modules *somehow*. And we're bound to be wrong in 50% of these cases. That's why I don't think it's a good idea.

That difference in highlighting wouldn't convey much information anyway: uppercase text with underscores is easy to notice as it is.

Fair enough :-)