|
From: | pnorcks |
Subject: | Re: Reorganize language files and add a new \language command. (issue2699041) |
Date: | Thu, 28 Oct 2010 05:15:57 +0000 |
Looks pretty good to me. Just a tiny style nitpick indicated below. Thanks for your work on this! Regards, Patrick http://codereview.appspot.com/2699041/diff/33001/input/regression/note-names.ly File input/regression/note-names.ly (right): http://codereview.appspot.com/2699041/diff/33001/input/regression/note-names.ly#newcode20 input/regression/note-names.ly:20: (set! pitchnames(ly:assoc-get 'nederlands language-pitch-names)) There should be a space between "pitchnames" and "(ly:assoc-get" http://codereview.appspot.com/2699041/
[Prev in Thread] | Current Thread | [Next in Thread] |