Issue 4356: Part combiner: generate mark events in scheme rather than C++

[Dan Eble, 2015-04-27 01:21:19]

On 2015/04/25 03:29:18, Dan Eble wrote:

Recent discussion with Keith on the devel list makes me think this should be
changed to work without requiring a new type of context.

[Dan Eble, 2015-04-28 02:29:44]

Eliminate new kind of context

[Dan Eble, 2015-04-28 03:02:49]

Eliminate unnecessary nesting

[dak, 2015-05-02 17:43:25]

https://codereview.appspot.com/228700043/diff/40001/ly/music-functions-init.ly
File ly/music-functions-init.ly (right):

https://codereview.appspot.com/228700043/diff/40001/ly/music-functions-init.l...
ly/music-functions-init.ly:1135: \context Voice = "one" \with {
#one-context-settings } {}
Since you are using the arguments *-context-settings in a context mod anyway, it
does not seem to make sense to pass them as music:

https://codereview.appspot.com/228700043/diff/40001/ly/music-functions-init.l...
ly/music-functions-init.ly:1153: #{ \voiceOne #}
Shouldn't this rather be #{ \with \voiceOne #} instead?

Or the equivalent?

[Dan Eble, 2015-05-02 18:22:14]

add \with as David suggests

[Dan Eble, 2015-05-04 01:34:29]

rebase

[Dan Eble, 2015-05-04 01:34:51]

rebase