----- Original Message -----
From: <d...@gnu.org>
To: <philehol...@googlemail.com>; <gra...@percival-music.ca>;
<tdanielsmu...@googlemail.com>; <pkx1...@gmail.com>
Cc: <re...@codereview-hr.appspotmail.com>; <lilypond-devel@gnu.org>
Sent: Monday, February 06, 2012 1:28 PM
Subject: Re: Stops woodwind diagram info appearing on terminal (issue
5636048)
http://codereview.appspot.com/5636048/diff/1/Documentation/snippets/new/woodwind-diagrams-key-lists.ly
File Documentation/snippets/new/woodwind-diagrams-key-lists.ly (right):
http://codereview.appspot.com/5636048/diff/1/Documentation/snippets/new/woodwind-diagrams-key-lists.ly#newcode16
Documentation/snippets/new/woodwind-diagrams-key-lists.ly:16:
#(print-keys-verbose 'piccolo (current-error-port))
This is ugly and tiresome to change. Why don't you wrap the whole
kaboodle in
#(with-output-to-port (current-error-port)
(lambda ()
(print-keys-verbose 'flute)
(print-keys-verbose 'flute-b-extension)
...
I'd probably even do a for-each around the various instruments, but then
the snippet would not really look
all that instructive anymore.
http://codereview.appspot.com/5636048/
I didn't do that because I had no idea it would work. As a snippet, I think
it's easier to leave as is because the average user (e.g. me) would not
understand the lambda stuff.
--
Phil Holmes
_______________________________________________
lilypond-devel mailing list
lilypond-devel@gnu.org
https://lists.gnu.org/mailman/listinfo/lilypond-devel
