Re: Add the command \offset to LilyPond (issue 8647044)

2013-10-06 Thread dak
Just before I forget: this is not a real comment on this issue. David N posted some update on this issue maybe a week ago (details are not all that important) after it lay dormant for months. I replied to that issue, and then figured I apparently didn't properly reply since Rietveld flagged "unp

Re: Add support for setting MIDI balance, pan position, reverb and chorus levels (issue 14434043)

2013-10-06 Thread ht . lilypond . development
On 2013/10/06 02:23:00, Keith wrote: Thanks, Heikki; this looks very good. Panning will help people to proof-read complex scores by listening to the MIDI output. I added an item to the bug tracker to add some documentation when this is done. Thanks. If there's any additional information or

Re: Add support for setting MIDI balance, pan position, reverb and chorus levels (issue 14434043)

2013-10-06 Thread k-ohara5a5a
On 2013/10/06 18:58:33, ht.lilypond.development wrote: Initially, I even tried to make Staff_performer listen to the SetProperty events directly, but failed because I couldn't then get the performer to acknowledge the new audio elements that it had announced itself. I see. What I was expec

Re: Move New_dynamic_engraver over the unused Dynamic_engraver (issue 14460043)

2013-10-06 Thread thomasmorley65
Although, I can't review C++, I've applied the patch for testing (hopefully without mistake) Testing this code { c''1_\mf^\> \break d''_\mp^\! } I've got: programming error: Spanner `Hairpin' is not fully contained in parent spanner. Ignoring orphaned part { c''1_\mf ^\> \break d''_

Re: Move New_dynamic_engraver over the unused Dynamic_engraver (issue 14460043)

2013-10-06 Thread dak
On 2013/10/06 23:00:25, thomasmorley651 wrote: Although, I can't review C++, I've applied the patch for testing (hopefully without mistake) Testing this code { c''1_\mf^\> \break d''_\mp^\! } I've got: programming error: Spanner `Hairpin' is not fully contained in parent spanner.

Re: Move New_dynamic_engraver over the unused Dynamic_engraver (issue 14460043)

2013-10-06 Thread thomasmorley65
On 2013/10/06 23:10:16, dak wrote: On 2013/10/06 23:00:25, thomasmorley651 wrote: [...] > Or am I completely wrong and this patch has nothing to do with the problem > above? In this case, you are completely wrong. [...] Anyway, thanks for clarifying. https://codereview.appspot.com/144600

Re: Move New_dynamic_engraver over the unused Dynamic_engraver (issue 14460043)

2013-10-06 Thread dak
On 2013/10/06 23:29:23, thomasmorley651 wrote: On 2013/10/06 23:10:16, dak wrote: > On 2013/10/06 23:00:25, thomasmorley651 wrote: [...] > > Or am I completely wrong and this patch has nothing to do with the problem > > above? > > In this case, you are completely wrong. [...] Anyway, thanks for