Re: Minor vim syntax improvements

[Graham Percival]

Why replace the line, though?  Wouldn't that stop it from finding
the other material that this line was looking at?

By the way, if anybody familiar with regex and/or the keywords
file and/or emacs and/or whatever, wants to jump in and say "yeah,
this looks fine", you'll save me a lot of feeling stupid and Jay a
lot of frustration.

Cheers,
- Graham


On Tue, 20 May 2008 18:44:20 -0700
"Jay Anderson" <address@hidden> wrote:

> If you look at dynamic-scripts-init.ly the forte dynamic is defined
> like this:
> 
> "f" = #(make-dynamic-script "f")
> 
> So when the keywords file is generated this very common dynamic was
> missing.
> 
> -----Jay
> 
> On Tue, May 20, 2008 at 12:44 AM, Graham Percival <address@hidden>
> wrote:
> > Thanks for the patch.
> >
> > I applied the changes to vim/lilypond-syntax.vim.  However, I
> > don't understand the change to buildscripts/lilypond-words.py, so I
> > did not apply it.
> >
> >
> > For anybody else interested: the change in question is replacing
> > line 39 of buildscripts/lilypond-words.py:
> > -    keywords += [w for w in re.findall (r"(?m)^\s*([a-zA-Z]+)
> > \s*=", s)]
> > +    keywords += [w for w in re.findall (r"(?m)^\s*\"?([a-zA-Z]+)\"?
> > \s*=", s)]
> >
> > Cheers,
> > - Graham
> >
> >
> > On Sat, 17 May 2008 16:36:53 -0700
> > "Jay Anderson" <address@hidden> wrote:
> >
> >> Attached is a patch which should make forte dynamic highlighted. It
> >> also fixes this: 'c--\mf'. Normally the second '-' is highlighted
> >> with the \mf which isn't right. Let me know if things seem ok
> >> (I've never submitted a patch before).
> >>
> >> -----Jay
> >>
> >