Papazian Christophe schreef:

* your patch makes a new spanner for every time_step it seems. Only make one if the name changes.

 * ly_scm2string() doesn't work for markup. Use scm_is_equal


I don't think that I make a new spanner every time_step. I use ly_scm2string to avoid such a behaviour. But you are right I must use scm_is_equal instead. Or did I miss something ?

Ok.

I'm also missing a change log and a .ly test file btw,

 * you will need a vector<Grob> for staves. You could have more than 1.


I don't understand. Each engraver is linked to only one staff. Why do I need
a vector now whereas I don't need one before ?


Piano Staff. it's a probably bug in the previous version as well.

--

Han-Wen Nienhuys - [EMAIL PROTECTED] - http://www.xs4all.nl/~hanwen

LilyPond Software Design
 -- Code for Music Notation
http://www.lilypond-design.com



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

Reply via email to