LGTM

https://codereview.appspot.com/547560044/diff/555240043/lily/spring.cc
File lily/spring.cc (right):

https://codereview.appspot.com/547560044/diff/555240043/lily/spring.cc#newcode119
lily/spring.cc:119: avg_stretch /= static_cast<Real> (springs.size ());
This is not an issue with this change, but should this function do
something other than divide by zero when springs is empty?

https://codereview.appspot.com/547560044/

Reply via email to