Re: Stops woodwind diagram info appearing on terminal (issue 5636048)

[Phil Holmes]

----- Original Message ----- 
From: <address@hidden>
To: <address@hidden>; <address@hidden>; 
<address@hidden>; <address@hidden>
Cc: <address@hidden>; <address@hidden>
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