[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: adding to the LSR
From: |
Reinhold Kainhofer |
Subject: |
Re: adding to the LSR |
Date: |
Wed, 29 Apr 2009 11:37:12 +0200 |
User-agent: |
KMail/1.11.2 (Linux/2.6.27-14-generic; KDE/4.2.2; i686; ; ) |
-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA1
On Dienstag, 28. April 2009 23:58:49 Carl D. Sorensen wrote:
> On 4/28/09 10:24 AM, "Jonathan Kulp" <address@hidden> wrote:
> > of my own files before. The convert-ly script took the \octaves command
> > as if it were an octave check instead of a user-defined macro. I've
> > changed the \octaves command in the original LSR code to \makeOctaves,
> > which will survive the convert-ly update intact.
>
> This probably also indicates a need to change the convert-ly rule for
> \octave. If it doesn't work for \octaves, it also wouldn't work for
> \octaveAdjustFunction, or some other user-defined variable that starts with
> \octave. This should be either fixed or added to the issue tracker to get
> fixed.
I think the solution is to simply check that no letter follows after \octave,
This can be easiest done by a negative lookahead patter in the RegExp:
str = re.sub (r"\\octave(?![a-zA-Z])", r"\\octaveCheck", str)
Patch is at:
http://codereview.appspot.com/53055
Okay to commit?
Notice that I only added the "(?![a-zA-Z])", which means: "Match the string
only if the pattern [a-zA-Z] does not match. The resulting match does only
contain the \octave, so the replacement string doesn't need to be changed.
> There are also some snippets that work but should be changed to reflect
> changes in 2.12.
Another is the X11 color list snippet.
> That's why I estimated 15 minutes per snippet (I wasn't thinking of
> automation, which I should have). We should check each of the snippets and
> see if the snippet is made obsolete or should be changed to reflect new
> features added, not just check to see if the syntax is right.
Yes, but first we'd need to get the LSR up and running with 2.12, then we can
update the snippets with new features and paradigm from 2.12.
Cheers,
Reinhold
- --
- ------------------------------------------------------------------
Reinhold Kainhofer, address@hidden, http://reinhold.kainhofer.com/
* Financial & Actuarial Math., Vienna Univ. of Technology, Austria
* http://www.fam.tuwien.ac.at/, DVR: 0005886
* LilyPond, Music typesetting, http://www.lilypond.org
-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.4.9 (GNU/Linux)
iD8DBQFJ+B/JTqjEwhXvPN0RAozLAKCpB4dYRdbzIZ1GWn8mPakq2bXoswCeMeKv
ia4q8K4orB8VsGWvVf0N75w=
=S+sl
-----END PGP SIGNATURE-----
- Re: adding to the LSR, (continued)
- Re: adding to the LSR, Jonathan Kulp, 2009/04/28
- Re: adding to the LSR, Carl D. Sorensen, 2009/04/28
- Re: adding to the LSR, Jonathan Kulp, 2009/04/28
- Re: adding to the LSR, Carl D. Sorensen, 2009/04/28
- Re: adding to the LSR, Chip, 2009/04/29
- Re: adding to the LSR, Graham Percival, 2009/04/29
- Re: adding to the LSR, Mats Bengtsson, 2009/04/29
- Re: adding to the LSR, Graham Percival, 2009/04/29
- Re: adding to the LSR, Jonathan Kulp, 2009/04/29
- Re: adding to the LSR, Jonathan Kulp, 2009/04/29
- Re: adding to the LSR,
Reinhold Kainhofer <=
- Re: adding to the LSR, Graham Percival, 2009/04/29
- Re: adding to the LSR, Reinhold Kainhofer, 2009/04/29
- Re: adding to the LSR, Valentin Villenave, 2009/04/29
- Re: adding to the LSR, Reinhold Kainhofer, 2009/04/29
- Re: adding to the LSR, Neil Puttock, 2009/04/28