Re: [Patch] Fix for Issue 879

2011-01-07 Thread Carl Sorensen
On 1/7/11 8:28 AM, "Phil Holmes" wrote: > I believe the attached patch fixes issue 879. Don't think it's a big deal, > but it was flagged as a frog task so I thought I'd contribute. I have run > my own regtest checker against the regtests and there w

[Patch] Fix for Issue 879

2011-01-07 Thread Phil Holmes
I believe the attached patch fixes issue 879. Don't think it's a big deal, but it was flagged as a frog task so I thought I'd contribute. I have run my own regtest checker against the regtests and there were no differences. Please could someone with git access check and

Re: Issue 879

2011-01-05 Thread Phil Holmes
- Original Message - From: "Keith OHara" To: Sent: Tuesday, January 04, 2011 7:40 PM Subject: Re: Issue 879 Phil Holmes philholmes.net> writes: \remove "Hara_kiri_engraver" \consists "Hara_kiri_engraver" I know I don't properly unders

Re: Issue 879

2011-01-04 Thread Keith OHara
Phil Holmes philholmes.net> writes: > \remove "Hara_kiri_engraver" > \consists "Hara_kiri_engraver" > > I know I don't properly understand this stuff still, but doesn't the > \consists re-instate the previously removed engraver on the line above? > You are correct. I looked at the history

Issue 879

2011-01-03 Thread Phil Holmes
Is about knees, crossing staves, and the RemoveEmptyStaff command. It took me to engraver-init.ly where I found: RemoveEmptyStaves = \with { \remove "Axis_group_engraver" \remove "Hara_kiri_engraver" \consists "Hara_kiri_engraver" \override Beam #'auto-knee-gap = #'() \override VerticalAx