[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: Move \RemoveEmptyStaves to new file for context modifications (issue
From: |
reinhold . kainhofer |
Subject: |
Re: Move \RemoveEmptyStaves to new file for context modifications (issue #1760) (issue4664076) |
Date: |
Wed, 13 Jul 2011 07:49:31 +0000 |
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/