[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 push if it's OK

Re: [Patch] Fix for Issue 879

2011-01-07 Thread Carl Sorensen
On 1/7/11 8:28 AM, Phil Holmes m...@philholmes.net 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 were no differences. Please

Re: Issue 879

2011-01-05 Thread Phil Holmes
- Original Message - From: Keith OHara k-ohara5...@oco.net To: lilypond-devel@gnu.org Sent: Tuesday, January 04, 2011 7:40 PM Subject: Re: Issue 879 Phil Holmes mail at philholmes.net writes: \remove Hara_kiri_engraver \consists Hara_kiri_engraver I know I don't properly

Re: Issue 879

2011-01-04 Thread Keith OHara
Phil Holmes mail at 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 of

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