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

Reply via email to