Looks good to me -- just a comment on a variable name.


http://codereview.appspot.com/4426072/diff/1001/lily/beam.cc
File lily/beam.cc (right):

http://codereview.appspot.com/4426072/diff/1001/lily/beam.cc#newcode1272
lily/beam.cc:1272: Interval vorboten;
We shouldn't use german words for variable names, I think.  English is
the standard code language for lilypond.

And if we do, we should spell it properly -- I'm pretty sure the
spelling is verboten.

Perhaps "disallowed"?

http://codereview.appspot.com/4426072/

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

Reply via email to