Re: Clean up inconsistencies in engraver-init.ly and performer-init.ly (issue 199460043 by thomasmorle...@gmail.com)

2015-02-18 Thread c_sorensen
On 2015/02/09 23:46:45, thomasmorley651 wrote: I'd happily do it, though, why alpabetically? And why only for the lines with \accept, could be done for the context-defs as well. And it's also thinkable to sort it following context-hierarchy: first bottom-contexts like Voice, then Staff up to

Re: Clean up inconsistencies in engraver-init.ly and performer-init.ly (issue 199460043 by thomasmorle...@gmail.com)

2015-02-18 Thread dak
On 2015/02/18 18:53:03, c_sorensen wrote: On 2015/02/09 23:46:45, thomasmorley651 wrote: I'd happily do it, though, why alpabetically? And why only for the lines with \accept, could be done for the context-defs as well. And it's also thinkable to sort it following context-hierarchy:

Re: Clean up inconsistencies in engraver-init.ly and performer-init.ly (issue 199460043 by thomasmorle...@gmail.com)

2015-02-18 Thread c_sorensen
On 2015/02/18 18:57:21, dak wrote: On 2015/02/18 18:53:03, c_sorensen wrote: On 2015/02/09 23:46:45, thomasmorley651 wrote: I'd happily do it, though, why alpabetically? And why only for the lines with \accept, could be done for the context-defs as well. And it's also thinkable to

Re: Clean up inconsistencies in engraver-init.ly and performer-init.ly (issue 199460043 by thomasmorle...@gmail.com)

2015-02-09 Thread thomasmorley65
On 2015/02/09 06:09:34, lemzwerg wrote: LGTM. It would be nice if David's checker script could be added, too. More, I'd consider it a good idea to use the checker on the current patch again. There was such a mess that I'm not sure I corrected all.

Re: Clean up inconsistencies in engraver-init.ly and performer-init.ly (issue 199460043 by thomasmorle...@gmail.com)

2015-02-08 Thread thomasmorley65
please review https://codereview.appspot.com/199460043/ ___ lilypond-devel mailing list lilypond-devel@gnu.org https://lists.gnu.org/mailman/listinfo/lilypond-devel

Re: Clean up inconsistencies in engraver-init.ly and performer-init.ly (issue 199460043 by thomasmorle...@gmail.com)

2015-02-08 Thread lemzwerg
LGTM. It would be nice if David's checker script could be added, too. https://codereview.appspot.com/199460043/diff/20001/ly/performer-init.ly File ly/performer-init.ly (right): https://codereview.appspot.com/199460043/diff/20001/ly/performer-init.ly#newcode174 ly/performer-init.ly:174: