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/

_______________________________________________
lilypond-devel mailing list
lilypond-devel@gnu.org
https://lists.gnu.org/mailman/listinfo/lilypond-devel

Reply via email to