[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: Minor vim syntax improvements
From: |
Graham Percival |
Subject: |
Re: Minor vim syntax improvements |
Date: |
Tue, 20 May 2008 19:58:44 -0700 |
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
> >>
> >