Hi Carl,

LGTM, apart from some indentation issues.

One question for you though: do you think the refactoring is an
improvement on the current code?  You could for example achieve the same
result with the following:

    (sort string-fret-fingering-tuples
          (lambda (a b) (> (car a) (car b)))))
    ;; end of determine-frets-and-strings

Cheers,
Neil



http://codereview.appspot.com/4518045/

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

Reply via email to