Looks good! Thanks for posting it. A couple of comments included.
https://codereview.appspot.com/235090043/diff/1/input/regression/stencil-scale.ly File input/regression/stencil-scale.ly (right): https://codereview.appspot.com/235090043/diff/1/input/regression/stencil-scale.ly#newcode8 input/regression/stencil-scale.ly:8: its bounding box.)" I don't think information about stencil-flip should be put in this regtest, unless stencil-flip is also added to the regtest. The headers of the regtests should only discuss the particular regtest, in my opinion. https://codereview.appspot.com/235090043/diff/1/scm/stencil.scm File scm/stencil.scm (right): https://codereview.appspot.com/235090043/diff/1/scm/stencil.scm#newcode467 scm/stencil.scm:467: (is-absolute (if (memq head-raw Probably a nice touch, since absolute? is probably not really a predicate. https://codereview.appspot.com/235090043/diff/1/scm/stencil.scm#newcode667 scm/stencil.scm:667: @var{axis} is 0 for X-axis, 1 for Y-axis." Perhaps you could also use X and Y instead of 0 and 1 (see the code below where you use X) https://codereview.appspot.com/235090043/ _______________________________________________ lilypond-devel mailing list lilypond-devel@gnu.org https://lists.gnu.org/mailman/listinfo/lilypond-devel
