Looks like it works, but I think it can (and should) be simplified.  If
it can't, then LGTM.


http://codereview.appspot.com/6193043/diff/1/ly/performer-init.ly
File ly/performer-init.ly (right):

http://codereview.appspot.com/6193043/diff/1/ly/performer-init.ly#newcode49
ly/performer-init.ly:49: \consists "Staff_performer"
Do these \consists need to be added?  Does the \alias Staff bring in the
Staff_performer and the Key_performer from the Staff Performer_group?

http://codereview.appspot.com/6193043/diff/1/ly/performer-init.ly#newcode88
ly/performer-init.ly:88: \consists "Dynamic_performer"
Again, don't the performers from the Voice performer group get added
automatically by the \alias Voice?

http://codereview.appspot.com/6193043/

_______________________________________________
lilypond-devel mailing list
lilypond-devel@gnu.org
https://lists.gnu.org/mailman/listinfo/lilypond-devel

Reply via email to