On 31/07/11 23:35, Carl Sorensen wrote:
> On 7/31/11 4:17 PM, "Wols Lists" <antli...@youngman.org.uk> wrote:
>
>> On 31/07/11 22:00, Janek Warchoł wrote:
>>> 2011/7/31 Wols Lists <antli...@youngman.org.uk>:
>>>>
>>>> Assuming that it's okay and is applied, I've redone my docu patch.
>>>
>>> Uploaded to Rietveld. I see one trailing whitespace (after "By
>>> default the chords are"), also there should be two spaces after a
>>> period which ends sentence.
>>>
>>>> The snippet prints a four-bar phrase with just the standard chord
>>>> (to trap the regression that bit us this time :-), then sets the capoPitch
>>>> to print transposed chords for the next four bars, then sets capoVertical
>>>> to print chords one above the other for the last four bars.
>
> Do we need four bars? Why not just do three bars -- one with capoPitch '(),
> another with capoPitch set, and a third with capoVertical?
>
> We like to get examples and regtests as simple as can be.
>
New modified regtest attached. I've cut it down to one line, two bars
per section so six in total. (I know you want minimal, but cutting it
down to one note per section feels a bit two much, and two minims looked
naff.) The other thing is, the new regtest doesn't have N/C, which
doesn't seem to matter, but that was almost the first thing I thought
needed checking when I first saw the chord-mode documentation I based my
sample and reg-test on.
Personally I prefer my longer version, but I'm quite happy to leave it
to you which version you actually go for (this second patch should apply
*instead* *of* the first).
Cheers,
Wol
>From 7259e0952b29775e4ec93014b09814c8e22bd26f Mon Sep 17 00:00:00 2001
From: Wol <anth...@youngman.org.uk>
Date: Mon, 1 Aug 2011 15:19:46 +0100
Subject: [PATCH 5/5] Add regression test for guitar capos
---
input/regression/chord-capo.ly | 28 ++++++++++++++++++++++++++++
1 files changed, 28 insertions(+), 0 deletions(-)
create mode 100644 input/regression/chord-capo.ly
diff --git a/input/regression/chord-capo.ly b/input/regression/chord-capo.ly
new file mode 100644
index 0000000..789f47d
--- /dev/null
+++ b/input/regression/chord-capo.ly
@@ -0,0 +1,28 @@
+\version "2.14.0"
+
+\header{
+ texidoc="Properties capoPitch, capoVertical: display chordnames, suitably
+transposed for a guitar capo, either in a line or one above the other.
+"
+}
+
+<<
+ \new ChordNames \chordmode {
+ c1
+ g1
+ c1
+ g1
+ c1
+ g1
+ }
+ \chordmode {
+ c1
+ g1
+ \set ChordNames.capoPitch = #(ly:make-pitch 0 -2 -1/2)
+ c1
+ g1
+ \set ChordNames.capoVertical = ##t
+ c1
+ g1
+ }
+>>
--
1.7.3.4
_______________________________________________
lilypond-devel mailing list
lilypond-devel@gnu.org
https://lists.gnu.org/mailman/listinfo/lilypond-devel
