https://codereview.appspot.com/329140043/diff/1/lily/axis-group-interface.cc
File lily/axis-group-interface.cc (right):

https://codereview.appspot.com/329140043/diff/1/lily/axis-group-interface.cc#newcode991
lily/axis-group-interface.cc:991: .in_color (1, 0, 1));
It's likely mostly a matter of taste but I'd lean towards using 1.0 and
0.0 here since the arguments are of type Real and it makes it more
obvious we are indeed talking about a fully bright color.  Of course,
same with all other occurences.

Other than that, LGTM.

https://codereview.appspot.com/329140043/

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

Reply via email to