Re: Tablature: proper support for tie/slur- and tie/glissando-constellations (issue2191042)

[Carl . D . Sorensen]

LGTM.

However, I'm a bit nervous about putting bends as well into the
Tab_tie_follow_engraver.  Not that the engraver won't work, but that the
Tab_tie_follow_engraver won't be part of the documentation.

Currently, I view Scheme engravers as a way for users (and snippets) to
add engraver functionality, but not as an optimal way to add core
functionality.

I'm not asking you to change your code, but I'm trying to send up a
caution flag to see what others might say about it.

Thanks,

Carl



http://codereview.appspot.com/2191042/diff/17001/input/regression/tablature-tie-slur-glissando.ly
File input/regression/tablature-tie-slur-glissando.ly (right):

http://codereview.appspot.com/2191042/diff/17001/input/regression/tablature-tie-slur-glissando.ly#newcode1
input/regression/tablature-tie-slur-glissando.ly:1: \version "2.13.37"
2.13.38

http://codereview.appspot.com/2191042/