On 2011-08-27, at 04:20 , pkx1...@gmail.com wrote:

> passes make and reg tests
> 
> http://codereview.appspot.com/4923048/


You could clean up Skyline::distance by pulling lines 532-548 into their own 
function and letting Skyline::distance call it with different options:

if (horizon_padding != 0.0)
  {
    Skyline padded_this(. . .);
    Skyline padded_other(. . .);
    return unpadded_distance(padded_this, padded_other);
  }
else
  {
    return unpadded_distance(this, &other);
  }
-- 
Dan


_______________________________________________
lilypond-devel mailing list
lilypond-devel@gnu.org
https://lists.gnu.org/mailman/listinfo/lilypond-devel

Reply via email to