On Mar 12, 2011, at 2:49 PM, hanw...@gmail.com wrote: > > http://codereview.appspot.com/4237057/diff/1/lily/spanner.cc > File lily/spanner.cc (right): > > http://codereview.appspot.com/4237057/diff/1/lily/spanner.cc#newcode405 > lily/spanner.cc:405: Spanner::broken_spanner_index (Spanner const *sp) > On 2011/03/12 10:18:06, MikeSol wrote: >> On 2011/03/09 23:03:44, hanwenn wrote: >> > why not make it a real member funtcion? > >> Actually - sorry, I spoke too soon. I see that there's a function >> get_break_index. Could these two functions be merged? Is there a > reason that >> the two functions exist separately? If not, I'll merge them together. > > good point. Can you replace broken_spanner_index by get_break_index > everywhere? should be separate commit to go in before this one. If it > passes the regtest cleanly, does not need to be reviewed. >
Passes the regtests and pushed. _______________________________________________ lilypond-devel mailing list lilypond-devel@gnu.org http://lists.gnu.org/mailman/listinfo/lilypond-devel
