From: Phil Holmes <mail@philholmes.net>
Date: Fri, 7 Jan 2011 15:15:09 +0000 (+0000)
Subject: Fix for 879
X-Git-Tag: release/2.13.46-1~14
X-Git-Url: https://git.donarmstrong.com/?a=commitdiff_plain;h=d406ee9fd73ec9b7273da824b3123cbba03bc952;p=lilypond.git

Fix for 879

Fixes issue 879 by removing the \override Beam #'auto-knee-gap = #'() line
in the definition of RemoveEmptyStaves.  Also adds comment why a \remove
is follwed by a \consists.
---

diff --git a/ly/engraver-init.ly b/ly/engraver-init.ly
index f5c4eab9b2..83d313b4d1 100644
--- a/ly/engraver-init.ly
+++ b/ly/engraver-init.ly
@@ -462,9 +462,13 @@ printing of a single line of lyrics."
 
 RemoveEmptyStaves = \with {
   \remove "Axis_group_engraver"
+    % If RemoveEmptyStaves is called twice, two
+    % Hara_kiri_engravers would be added, which leads to a
+    % warning.
+    % This code makes sure that no previous Hara_kiri_engraver
+    % is left before adding a new one.
   \remove "Hara_kiri_engraver"
   \consists "Hara_kiri_engraver"
-  \override Beam #'auto-knee-gap = #'()
   \override VerticalAxisGroup #'remove-empty = ##t
 }