Re: Make test-output-distance regtest contain span bars (issue 7400057)

2013-03-05 Thread dak
On 2013/03/02 07:30:11, Keith wrote: LGTM. On 2013/02/28 05:57:45, dak wrote: I think we have a special span bar at the start, The bar at the start is a SystemStartBar, which was not affected by the recent span-bar bug. You might want an extra measure, and a final bar, then. Also,

Re: Make test-output-distance regtest contain span bars (issue 7400057)

2013-03-01 Thread k-ohara5a5a
LGTM. On 2013/02/28 05:57:45, dak wrote: I think we have a special span bar at the start, The bar at the start is a SystemStartBar, which was not affected by the recent span-bar bug. You might want an extra measure, and a final bar, then. Also, some note in the texidoc saying that this test

Make test-output-distance regtest contain span bars (issue 7400057)

2013-02-27 Thread lemzwerg
LGTM. https://codereview.appspot.com/7400057/ ___ lilypond-devel mailing list lilypond-devel@gnu.org https://lists.gnu.org/mailman/listinfo/lilypond-devel

Re: Make test-output-distance regtest contain span bars (issue 7400057)

2013-02-27 Thread thomasmorley65
LGTM One suggestion: https://codereview.appspot.com/7400057/diff/1/input/regression/test-output-distance.ly File input/regression/test-output-distance.ly (right): https://codereview.appspot.com/7400057/diff/1/input/regression/test-output-distance.ly#newcode20

Re: Make test-output-distance regtest contain span bars (issue 7400057)

2013-02-27 Thread dak
Reviewers: lemzwerg, thomasmorley65, Message: On 2013/02/28 00:45:57, thomasmorley65 wrote: LGTM One suggestion: https://codereview.appspot.com/7400057/diff/1/input/regression/test-output-distance.ly File input/regression/test-output-distance.ly (right):