[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: Fix 1805: AmbitusAccidental needs avoid-slur, needed when the notes
From: |
tdanielsmusic |
Subject: |
Re: Fix 1805: AmbitusAccidental needs avoid-slur, needed when the notes in the ambitus have a slur (issue 4904049) |
Date: |
Wed, 07 Sep 2011 07:41:14 +0000 |
LGTM, but the regession test needs work
http://codereview.appspot.com/4904049/diff/3001/input/regression/ambitus-slur.ly
File input/regression/ambitus-slur.ly (right):
http://codereview.appspot.com/4904049/diff/3001/input/regression/ambitus-slur.ly#newcode3
input/regression/ambitus-slur.ly:3: texidoc = "Ambitus also works with
slurs.
plus "so no misleading warnings are issued."
http://codereview.appspot.com/4904049/diff/3001/input/regression/ambitus-slur.ly#newcode10
input/regression/ambitus-slur.ly:10: c'4( e')
On 2011/09/05 15:55:30, Neil Puttock wrote:
need an accidentalled note here
It's not necessary to produce the unwanted error
http://codereview.appspot.com/4904049/