LGTM

http://codereview.appspot.com/4339053/diff/1/lily/beam-collision-engraver.cc
File lily/beam-collision-engraver.cc (right):

http://codereview.appspot.com/4339053/diff/1/lily/beam-collision-engraver.cc#newcode60
lily/beam-collision-engraver.cc:60: break;
you can do

for ..
  if (covered->has_interface(..))
    return true

return false

http://codereview.appspot.com/4339053/diff/1/scm/define-grob-properties.scm
File scm/define-grob-properties.scm (right):

http://codereview.appspot.com/4339053/diff/1/scm/define-grob-properties.scm#newcode174
scm/define-grob-properties.scm:174: which a beam should not collide.")
more exact: list of ifaces for which automatic beam-collision
resolution is run.

http://codereview.appspot.com/4339053/

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

Reply via email to