Re: make pretty-print available in ly files (issue 222810043 by david.nales...@gmail.com)

2016-01-23 Thread david . nalesnik
On 2016/01/23 22:11:03, david.nalesnik wrote: The patch was pushed after much of the discussion above. Shall I make another patch removing the functionality> Ugh--I mean pushed *before* much of the discussion. https://codereview.appspot.com/222810043/

Re: make pretty-print available in ly files (issue 222810043 by david.nales...@gmail.com)

2016-01-23 Thread david . nalesnik
The patch was pushed after much of the discussion above. Shall I make another patch removing the functionality> https://codereview.appspot.com/222810043/ ___ lilypond-devel mailing list lilypond-devel@gnu.org

Re: make pretty-print available in ly files (issue 222810043 by david.nales...@gmail.com)

2015-04-08 Thread dak
On 2015/04/08 09:13:53, thomasmorley651 wrote: We have 'write-me' in lily-library.scm, pretty-print is loaded there anyway. We could change 'write-me' to something at the lines of: (define*-public (write-me x #:optional (message )) Return @var{x}. Display @var{message} and write

Re: make pretty-print available in ly files (issue 222810043 by david.nales...@gmail.com)

2015-04-08 Thread thomasmorley65
On 2015/04/08 10:21:55, dak wrote: value-lily-string is probably the most useful reasonably generic printing facility right now. But nobody really knows it. At least, I wasn't aware of it, because of (in a ly-file) #(display value-lily-string) - Unbound variable: value-lily-string

Re: make pretty-print available in ly files (issue 222810043 by david.nales...@gmail.com)

2015-04-08 Thread dak
On 2015/04/08 10:43:09, thomasmorley651 wrote: On 2015/04/08 10:21:55, dak wrote: value-lily-string is probably the most useful reasonably generic printing facility right now. But nobody really knows it. At least, I wasn't aware of it, because of (in a ly-file) #(display

Re: make pretty-print available in ly files (issue 222810043 by david.nales...@gmail.com)

2015-04-08 Thread dak
On 2015/04/07 23:25:09, thomasmorley651 wrote: https://codereview.appspot.com/222810043/diff/1/ly/init.ly File ly/init.ly (right): https://codereview.appspot.com/222810043/diff/1/ly/init.ly#newcode38 ly/init.ly:38: #(use-modules (ice-9 pretty-print)) On 2015/04/07 17:43:07, david.nalesnik

Re: make pretty-print available in ly files (issue 222810043 by david.nales...@gmail.com)

2015-04-08 Thread thomasmorley65
On 2015/04/08 08:11:35, dak wrote: On 2015/04/07 23:25:09, thomasmorley651 wrote: https://codereview.appspot.com/222810043/diff/1/ly/init.ly File ly/init.ly (right): https://codereview.appspot.com/222810043/diff/1/ly/init.ly#newcode38 ly/init.ly:38: #(use-modules (ice-9 pretty-print)) On

Re: make pretty-print available in ly files (issue 222810043 by david.nales...@gmail.com)

2015-04-08 Thread david . nalesnik
On 2015/04/08 11:14:54, dak wrote: On 2015/04/08 10:43:09, thomasmorley651 wrote: On 2015/04/08 10:21:55, dak wrote: value-lily-string is probably the most useful reasonably generic printing facility right now. But nobody really knows it. At least, I wasn't aware of it, because of (in

Re: make pretty-print available in ly files (issue 222810043 by david.nales...@gmail.com)

2015-04-08 Thread david . nalesnik
On 2015/04/08 16:06:47, david.nalesnik wrote: On 2015/04/08 11:14:54, dak wrote: commit 8d8e8aec6388fbb08ed2219884b82ecf53a9dbcd Author: David Kastrup mailto:d...@gnu.org Date: Wed Jan 1 20:54:15 2014 +0100 Provide value-lily-string function because I really needed that facility

Re: make pretty-print available in ly files (issue 222810043 by david.nales...@gmail.com)

2015-04-07 Thread dak
https://codereview.appspot.com/222810043/diff/1/ly/init.ly File ly/init.ly (right): https://codereview.appspot.com/222810043/diff/1/ly/init.ly#newcode38 ly/init.ly:38: #(use-modules (ice-9 pretty-print)) Uh, where is the point? Why wouldn't a user include a module he wants himself? Other

Re: make pretty-print available in ly files (issue 222810043 by david.nales...@gmail.com)

2015-04-07 Thread david . nalesnik
https://codereview.appspot.com/222810043/diff/1/ly/init.ly File ly/init.ly (right): https://codereview.appspot.com/222810043/diff/1/ly/init.ly#newcode38 ly/init.ly:38: #(use-modules (ice-9 pretty-print)) On 2015/04/07 16:13:44, dak wrote: Uh, where is the point? Why wouldn't a user include a

Re: make pretty-print available in ly files (issue 222810043 by david.nales...@gmail.com)

2015-04-07 Thread thomasmorley65
https://codereview.appspot.com/222810043/diff/1/ly/init.ly File ly/init.ly (right): https://codereview.appspot.com/222810043/diff/1/ly/init.ly#newcode38 ly/init.ly:38: #(use-modules (ice-9 pretty-print)) On 2015/04/07 17:43:07, david.nalesnik wrote: On 2015/04/07 16:13:44, dak wrote: Uh,

make pretty-print available in ly files (issue 222810043 by david.nales...@gmail.com)

2015-03-27 Thread david . nalesnik
Reviewers: , Message: Please review. Thanks! Description: make pretty-print available in ly files Please review this at https://codereview.appspot.com/222810043/ Affected files (+1, -0 lines): M ly/init.ly Index: ly/init.ly diff --git a/ly/init.ly b/ly/init.ly index