Re: Add tab-tie-follow-engraver (issue2723043)

[Carl . D . Sorensen]

Thanks for the help on the null pointer.  I was thinking that it was
some *other* kind of variable than a Grob, and that I was casting it
wrong.

Got that all taken care of -- no scheme calls at all.

On 2010/11/12 17:40:42, Neil Puttock wrote:

> It's back to square one though, isn't it?

> The triple nested loop is horrible, and should be avoided at all
costs; this is
> partly why I suggested only acknowledging the ties and heads.

OK, so now I've eliminated the triple nested loop.

There is what appears to me to be a required nested loop.

One loop to loop through the note-heads.
Then an inner loop (with a break) through the ties looking for a tie on
a note head.
Followed by a second inner loop (with a break) through the slurs looking
for a slur on the note head.
Followed by a third inner loop (if we didn't find a slur) through the
glissandos looking for a glissando on the note head.

I can think of no way to simplify this code further.  If you have any
ideas I'd be happy to hear them.

Thanks,

Carl


http://codereview.appspot.com/2723043/