Re: Staff.instr code change?

[Han-Wen Nienhuys]

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 - address@hidden - http://www.xs4all.nl/~hanwen

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