Re: Adds beam collision engraver (issue4128059)

[Mike Solomon]

On Feb 7, 2011, at 10:00 AM, Han-Wen Nienhuys wrote:

On Mon, Feb 7, 2011 at 12:43 PM, Mike Solomon <address@hidden> wrote:

Yikes!  Sorry...

http://codereview.appspot.com/4080057 is the real deal.

This still adds noteheads from the same voice as the beam.

+ for (vsize i=0; i < covered_grobs_.size (); i++)

+ for (vsize j=0; j < active_beams_.size (); j++)

+ {

+ bool my_beam = false;

+ if (Grob *stem = unsmob_grob (covered_grobs_.at (i)->get_object ("stem")))

+ if (Grob *beam = unsmob_grob (stem->get_object ("beam")))

+ if (beam == active_beams_.at (j))

+ my_beam = true;

+ if (!my_beam)

+ Pointer_group_interface::add_grob (active_beams_.at (j), ly_symbol2scm ("covered-grobs"), covered_grobs_.at (i));

+ }

I thought that did the trick...am I missing something?

Cheers,

MS