Re: Fix collisions between hairpins and dynamic text spanners.

[n . puttock]

Reviewers: Reinhold,

Message:
On 2009/09/01 11:41:15, Reinhold wrote:
> LGTM. But a regtest is missing ;-)

Ah, that was just me being lazy. :)

I'll add a regtest when I commit the patch.

I'm wondering whether adjacent-hairpins needs a convert rule.  I'm
thinking not, since it's an internal property, but there may be the odd
adventurous user using it for some advanced tweaks.

Any thoughts?

Regards,
Neil



Description:
Fix collisions between hairpins and dynamic text spanners.

* rename 'adjacent-hairpins to 'adjacent-spanners and add
DynamicTextSpanner
  grobs in New_dynamic_engraver as well as hairpins

* in Hairpin::print (), use renamed property to catch text spans and
apply
  the appropriate amount of bound-padding

Please review this at http://codereview.appspot.com/110122

Affected files:
  M lily/hairpin.cc
  M lily/new-dynamic-engraver.cc
  M scm/define-grob-properties.scm