http://codereview.appspot.com/5440084/diff/7001/lily/parser.yy File lily/parser.yy (right):
http://codereview.appspot.com/5440084/diff/7001/lily/parser.yy#newcode432 lily/parser.yy:432: %type <scm> list_music I must *strongly* recommend that the name of either music_list or list_music be changed. Even if the names make distinct sense, it is far to easy to transpose identifiers like this when reading or writing code. (I have made this mistake in my own code many times in the past.) Given the existence of other _list types, I suggest that the name of list_music be changed. Maybe "wrapped_music" or "music_chord"... http://codereview.appspot.com/5440084/ _______________________________________________ lilypond-devel mailing list lilypond-devel@gnu.org https://lists.gnu.org/mailman/listinfo/lilypond-devel
