https://codereview.appspot.com/201140043/diff/20001/lily/spanner.cc File lily/spanner.cc (right):
https://codereview.appspot.com/201140043/diff/20001/lily/spanner.cc#newcode404 lily/spanner.cc:404: Rod r; Perhaps we should call this rod something else to underscore that it's not the same rod as the one declared above. Sure, it's fine because the last r was declared in a nested scope and "'r' is for rod!", but this is unnecessarily confusing (even after moving the declaration from where it was originally, which was really obscure). https://codereview.appspot.com/201140043/ _______________________________________________ lilypond-devel mailing list lilypond-devel@gnu.org https://lists.gnu.org/mailman/listinfo/lilypond-devel