http://codereview.appspot.com/4974075/diff/1/lily/ledger-line-spanner.cc File lily/ledger-line-spanner.cc (right):
http://codereview.appspot.com/4974075/diff/1/lily/ledger-line-spanner.cc#newcode362 lily/ledger-line-spanner.cc:362: ledgers.add_stencil (ledger_line); Since you no longer seem to be using the brew_ledger_lines function, please get rid of it. http://codereview.appspot.com/4974075/diff/1/lily/staff-symbol.cc File lily/staff-symbol.cc (right): http://codereview.appspot.com/4974075/diff/1/lily/staff-symbol.cc#newcode82 lily/staff-symbol.cc:82: int l = line_positions.size (); Could you use a different name? l looks an awful lot like 1... http://codereview.appspot.com/4974075/diff/1/lily/staff-symbol.cc#newcode105 lily/staff-symbol.cc:105: int l = scm_to_int (scm_length (line_positions)); and again... http://codereview.appspot.com/4974075/diff/1/lily/staff-symbol.cc#newcode117 lily/staff-symbol.cc:117: int l = Staff_symbol::line_count (me); ok, this one isn't your fault, but I still think it would be good to change it. http://codereview.appspot.com/4974075/ _______________________________________________ lilypond-devel mailing list lilypond-devel@gnu.org https://lists.gnu.org/mailman/listinfo/lilypond-devel