#4657 Tweak Rest.staff-position fails with beam

[Simon Albrecht]

If a beam is present, tweaking Rest.staff-position remains without effect:

:::TeX
%%% tweak-rest-staff-position-with-beam.ly
\version "2.19.28"
{
  \voiceTwo
  f'16[ \tweak staff-position 10 r16 16]
}

(Same problem with \once\override)
[EDIT: No, it’s not. Overriding works fine.]

[David Kastrup]

I don't see "same problem with \once override": overrides seem to work fine here.

But the tweak indeed appears to miss its mark.

[Simon Albrecht]

Indeed, I don’t know why I wrote that. Probably I meant ‘Not an issue with \once\override’ anyway.

[David Kastrup]

Issue 4657: Tweak Rest.staff-position fails with beam

The problem here appears to be that Beam::rest-collision-callback
returned 0 instead of any previously established offset when it
decided not to tamper with the default rest positioning without beam.

This also rewrites Beam::pure-rest-collision-callback (which appears
to have dealt correctly with the offsets here and appears
disconcertingly diverged from the unpure callback) a bit.

http://codereview.appspot.com/275320043

[Anonymous]

Passes make, make check and and a full make doc.

[Anonymous]

Patch on countdown for November 19th.

[Anonymous]

Patch counted down - please push.

[David Kastrup]

Pushed to staging as
commit e8ec0112dfc34f21d01fb42538b7ce1b895fd9cc
Author: David Kastrup dak@gnu.org
Date: Fri Nov 13 17:54:46 2015 +0100

Issue 4657: Tweak Rest.staff-position fails with beam

The problem here appears to be that Beam::rest-collision-callback
returned 0 instead of any previously established offset when it
decided not to tamper with the default rest positioning without beam.

This also rewrites Beam::pure-rest-collision-callback (which appears
to have dealt correctly with the offsets here and appears
disconcertingly diverged from the unpure callback) a bit.