Add regtests for numbered and single-digit time signature styles

http://codereview.appspot.com/162300043

Patchy the autobot says: passes tests. Includes a full make doc

Patch on countdown for November 2nd

Patch counted down - please push

Git patch attached. TIA.

author Dan Eble
Mon, 27 Oct 2014 02:24:35 +0000 (22:24 -0400) committer James Lowe
Wed, 5 Nov 2014 11:26:47 +0000 (11:26 +0000) commit 586a6d331e22d0e1a255068f07c534d219c7c902

Two bugs, issue 4178 and issue 4179, were discovered while developing these tests, showing that extremely long measures exceed what LilyPond is prepared to handle.

After discovering these issues, we seem to have pushed these regression tests anyway.

Probably three-digit numbers would suffice to test the printing of time signatures. \time 1/128 d128 \time 101 d1*101

Comment #7 should have referenced issue 4179 and issue 4180 above, and new issue 4231. Let's change the two regression tests added above to :

\header { texidoc = "The numbered time signature style prints a fraction." }

\new Staff { \relative d' { \override Staff.TimeSignature.style = #'numbered \time 4/4 d1 \time 3/4 d2. \time 2/2 d1 \time 1/128 d128 #(ly:expect-warning "strange time signature found: 10/6") \time 10/6 \tuplet 6/4 { d2. d2 d2. d2 } } }

\header { texidoc = "The single-digit time signature style prints the numerator only." }

\new Staff { \relative d' { \override Staff.TimeSignature.style = #'single-digit \time 1/2 d2 \time 2/4 d2 \time 3/4 d2. \time 8/128 d16 \time 123/1 d1*123 } }

I'm confused on this one, I'm guessing this should now have no patch label at this time until someone submits something for testing?

I didn't think that adding two new regtests needed any testing beyond what I can do, but I made a thingy http://codereview.appspot.com/193910043

Patchy the autobot says: passes tests. includes a full make doc

Patch on countdown for Jan 10th

Patch counted down - please push

Patch counted down - please push

Patch counted down - please push

commit 8298bc08d2d6398af3b1c988b825ad36d355e7ee