I read through this patch, but I can't tell what it does or is supposed to do. Maybe one of our pure gurus should look at this. Joe?
http://codereview.appspot.com/4917046/diff/1/lily/align-interface.cc File lily/align-interface.cc (right): http://codereview.appspot.com/4917046/diff/1/lily/align-interface.cc#newcode399 lily/align-interface.cc:399: Align_interface::vertical_sort (Grob *g1, Grob *g2) this doesn't sort. http://codereview.appspot.com/4917046/diff/1/lily/pure-from-neighbor-engraver.cc File lily/pure-from-neighbor-engraver.cc (right): http://codereview.appspot.com/4917046/diff/1/lily/pure-from-neighbor-engraver.cc#newcode4 lily/pure-from-neighbor-engraver.cc:4: Copyright (C) 1997--2011 Han-Wen Nienhuys <han...@xs4all.nl> ? http://codereview.appspot.com/4917046/diff/1/lily/pure-from-neighbor-engraver.cc#newcode53 lily/pure-from-neighbor-engraver.cc:53: && !i.grob ()->internal_has_interface (ly_symbol2scm ("pure-from-neighbor-interface"))) can't you put this in acknowledge_pure_from_neighbor() so you don't have to check the interface ? http://codereview.appspot.com/4917046/ _______________________________________________ lilypond-devel mailing list lilypond-devel@gnu.org https://lists.gnu.org/mailman/listinfo/lilypond-devel
