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

http://codereview.appspot.com/4290069/diff/5001/lily/beam-collision-engraver.cc#newcode55
lily/beam-collision-engraver.cc:55: if (covered_grobs_.size ())
I'm going to start testing this now, but it's sad to see:
- an 8-level nested if/loop construct (!(?(!(?(!)?)!)?)!)
- so many comments vanishing

Hopefully somebody can suggest a refactoring which is easier to read?

http://codereview.appspot.com/4290069/

_______________________________________________
lilypond-devel mailing list
[email protected]
http://lists.gnu.org/mailman/listinfo/lilypond-devel

Reply via email to