Re: Move \RemoveEmptyStaves to new file for context modifications (issue #1760) (issue4664076)

[reinhold . kainhofer]

LGTM.

On 2011/07/13 07:33:39, Janek Warchol wrote:
> The change looks pretty straightforward, but i don't understand why
simply
> moving RemoveEmptyStaff definition changes anything :(

Because ly/declarations-init.ly contains:

\layout {
    \include "engraver-init.ly"
}

So the whole contents of engraver-init.ly (including the old definition
of RemoveEmptyStaves) is interpreted and defined inside a layout block
and is thus not available at top-level, just inside any other layout
block...
This patch moves the definition out of the \layout block and thus makes
it available for use inside a score, to.

Cheers,
Reinhold


http://codereview.appspot.com/4664076/