LGTM just cosmetics.
http://codereview.appspot.com/4210051/diff/1/lily/stem.cc File lily/stem.cc (right): http://codereview.appspot.com/4210051/diff/1/lily/stem.cc#newcode336 lily/stem.cc:336: SCM s = ly_assoc_get (ly_symbol2scm ("lengths"), details, SCM_EOL); can you rename this for easier readability? http://codereview.appspot.com/4210051/diff/1/lily/stem.cc#newcode360 lily/stem.cc:360: Real shortening_step = min (max (0.25, (shorten_property / 6)), 0.5); shouldnt 6 be 2*staff_radius+1 ? http://codereview.appspot.com/4210051/diff/1/lily/stem.cc#newcode368 lily/stem.cc:368: which_step += abs(hp[dir]); can you collapse these 3 lines? http://codereview.appspot.com/4210051/ _______________________________________________ lilypond-devel mailing list lilypond-devel@gnu.org http://lists.gnu.org/mailman/listinfo/lilypond-devel