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.
>>
>> iirc that will then become the regression test for this feature?
> 
> Yes, a regression test should contain a similar snippet.  Could you
> add the regression test file to your commit?  (create a new file in
> input/regression, named appropriately, use 'git add
> input/regression/<your regtest filename>' to add it to what would be
> commited, and commit.
> 
Regression test attached. I've looked at the other regression tests and
tried to make it similar. Basically just put the header stuff above my
test/docu snippet.

Cheers,
Wol
From cf59539f59324583bbe595343e6ea3ca0c754791 Mon Sep 17 00:00:00 2001
From: Wol <anth...@youngman.org.uk>
Date: Sun, 31 Jul 2011 23:13:38 +0100
Subject: [PATCH 3/3] Add regression test for guitar capos

---
 input/regression/chord-capo.ly |   44 ++++++++++++++++++++++++++++++++++++++++
 1 files changed, 44 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..aa669e2
--- /dev/null
+++ b/input/regression/chord-capo.ly
@@ -0,0 +1,44 @@
+\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
+    r1
+    g1
+    c1
+    \break
+    c1
+    r1
+    g1
+    c1
+    \break
+    c1
+    r1
+    g1
+    c1
+  }
+  \chordmode {
+    c1
+    r1
+    g1
+    c1
+    \break
+    \set ChordNames.capoPitch = #(ly:make-pitch 0 -2 -1/2)
+    c1
+    r1
+    g1
+    c1
+    \break
+    \set ChordNames.capoVertical = ##t
+    c1
+    r1
+    g1
+    c1
+  }
+>>
-- 
1.7.3.4

_______________________________________________
lilypond-devel mailing list
lilypond-devel@gnu.org
https://lists.gnu.org/mailman/listinfo/lilypond-devel

Reply via email to