https://codereview.appspot.com/571180043/diff/565230043/ly/spanners-init.ly File ly/spanners-init.ly (right):
https://codereview.appspot.com/571180043/diff/565230043/ly/spanners-init.ly#newcode25 ly/spanners-init.ly:25: "View side-by-side diff with in-line comments" is broken for this file. https://codereview.appspot.com/571180043/diff/565230043/scm/define-grobs.scm File scm/define-grobs.scm (right): https://codereview.appspot.com/571180043/diff/565230043/scm/define-grobs.scm#newcode1457 scm/define-grobs.scm:1457: ;(font-size . -2) Mmh, this is commented. Why? Same below. https://codereview.appspot.com/571180043/diff/565230043/scm/define-music-types.scm File scm/define-music-types.scm (right): https://codereview.appspot.com/571180043/diff/565230043/scm/define-music-types.scm#newcode311 scm/define-music-types.scm:311: "View side-by-side diff with in-line comments" broken here as well https://codereview.appspot.com/571180043/diff/565230043/scm/scheme-engravers.scm File scm/scheme-engravers.scm (right): https://codereview.appspot.com/571180043/diff/565230043/scm/scheme-engravers.scm#newcode98 scm/scheme-engravers.scm:98: (define-public (Measure_attached_spanner_engraver context) Not related to the current patch: Meanwhile I've seen several scheme-spanners-engravers for new spanner-grobs (and wrote some for my own work) or to customize existing spanner-grobs. All are more or less the same, ofcourse they need to be so. I'm musing whether it would be possible to create some specialized macro. We already have `make-engraver´, maybe something like `make-spanner-engraver´. Thoughts? https://codereview.appspot.com/571180043/
