Hi,

sorry, Mike - i got confused and forgotten that a patchset newer than
what i had is here, on Rietveld.
I see that you've fixed hairpins and added skylines to accidentals,
flags and beams - great!
Unfortunately I've spotted some problems with DynamicSigns - they are
too high (as of patchset 21).  I'll send you pdf files privately.

As for compilation times, they are better in case of bigger scores:

dynamics.ly:
master 0m0.859s
previous skylines 0m1.520s
skylines from patchset 21: 0m2.994s

Eja-Mater.ly:
master 0m7.373s
previous skylines 0m13.331s
skylines from patchset 21: 0m11.387s

tota-pulchra.ly:
master 0m8.400s
previous skylines 0m15.649s
skylines from patchset 21: 0m15.649s

HTH,
Janek

http://codereview.appspot.com/5626052/

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

Reply via email to