On 2011/06/28 16:05:26, Reinhold wrote:
On 2011/06/28 15:03:40, J_lowe wrote:
> Passes reg tests - dynamics-alignment-breaker.ly is obviously
different as the
> reg test itself has been changed.

Yes, I removed that second \breakDynamicSpan, because I wanted to
check that
while the first cresc is broken, the second line spanner is really
left intact
and the decrescendo and the \p line up.
I'll remove the line break, though, because that's tested in the
*-linebreak.ly
test case

Reinhold,

I re-did the reg test, the only difference is the
'dynamics-alignment-breaker.ly' as I said before.

I haven't done a comparison with the difference between the last patch
and this one, only as base check,

It looks ok (again it is different as expected) but for completeness
(and because i cannot do it here) I have attached a png of the
differences on issues 1259 and 1433:

http://codereview.appspot.com/4630070/

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

Reply via email to