http://codereview.appspot.com/4446060/diff/3/lily/beam.cc
File lily/beam.cc (right):

http://codereview.appspot.com/4446060/diff/3/lily/beam.cc#newcode1208
lily/beam.cc:1208: if ((Beam::is_cross_staff (me)
On 2011/04/25 03:20:51, hanwenn wrote:
comment why we are doing this.

If we don't do this, this continues to be a problem:


<<
  \new Staff = up \relative c'' {
\time 2/4
\autoBeamOff
    b8 b b b
  }
  \new Staff = down \relative c'' {
    b b b b
    \change Staff = up
  }


as does this


<<
  \new Staff = up \relative c'' {
\time 2/4
\autoBeamOff
    b8 b b b
  }
  \new Staff = down \relative c'' {
    b b b b
    \change Staff = up
  }


as does this


<<
  \new Staff = up \relative c'' {
\time 2/4
    b8 b b b
  }
  \new Staff = down \relative c'' {
    b b b b
    \change Staff = up
  }


Granted, I could keep this in the engraver as it previously was, but I
was under the impression you wanted this logic in beam.cc.

http://codereview.appspot.com/4446060/

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

Reply via email to