On Sun, 2014-06-01 at 10:06 -0700, Evan Martin wrote:
> In misc/ninja-mode.el:
> 
> > @@ -31,6 +31,7 @@
> >         '("\\([[:alnum:]_]+\\) =" . (1 font-lock-variable-name-face))
> >         ;; Variable expansion.
> >         '("\\($[[:alnum:]_]+\\)" . (1 font-lock-variable-name-face))
> > +	   '("\\(${[[:alnum:]_]+}\\)" . (1 font-lock-variable-name-face))
> 
> (Sorry for the questions, my elisp isn't so good.)
> 
> Why indent extra here?
> 
> If you'd like to make the regex match Ninja's behavior, the difference
> between $foo and ${foo} is the difference between "simple_varname" and
> "varname" as defined here:
> https://github.com/martine/ninja/blob/master/src/lexer.in.cc#L132
> 
> —
> Reply to this email directly or view it on GitHub.
> 
The extra indent was my fault (I had a tab which looked like 4 spaces on
my machine). I fixed the regexp