Hi Harm, I took a quick look and it LGTM at a quick read. I have a couple nit-level suggestions. -Paul
https://codereview.appspot.com/347000043/diff/40001/scm/markup.scm File scm/markup.scm (right): https://codereview.appspot.com/347000043/diff/40001/scm/markup.scm#newcode141 scm/markup.scm:141: ;; The string is split at line-breaks, emty strings removed and finally typo: empty https://codereview.appspot.com/347000043/diff/40001/scm/markup.scm#newcode148 scm/markup.scm:148: simple-markup))) Indent 'list' expression so it is clearer that it is the second argument to 'member' and not another 'and' expression. https://codereview.appspot.com/347000043/ _______________________________________________ lilypond-devel mailing list lilypond-devel@gnu.org https://lists.gnu.org/mailman/listinfo/lilypond-devel
