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" <[EMAIL PROTECTED]> 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 <[EMAIL PROTECTED]> > 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" <[EMAIL PROTECTED]> 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 > >> > > _______________________________________________ bug-lilypond mailing list bug-lilypond@gnu.org http://lists.gnu.org/mailman/listinfo/bug-lilypond
