Re: [Lilypond-auto] Issue 3631 in lilypond: 2.17 does a worse job with vertical spacing and/or the page layout than 2.16

[lilypond]

Comment #7 on issue 3631 by address@hidden: 2.17 does a worse job with  
vertical spacing and/or the page layout than 2.16
http://code.google.com/p/lilypond/issues/detail?id=3631

Oh wow.  I tried the following:
\markup
\on-the-fly #(lambda (layout props arg)
              (for-each
               (lambda (ps)
                (format #t "~S\n" (ly:prob-property ps 'staff-refpoint-extent)))
               (vector->list (ly:paper-score-paper-systems
                              (ly:score-embedded-format
                               #{ \score {
                               \new Voice \repeat unfold 50
                               { c,,,4 \repeat unfold 30 r4 c''''''4 \repeat 
unfold 30 r4 }
                               \layout { }
                             } #} layout))))
              empty-stencil) \null

Now the relevant output for 2.16.2 and 2.17.14 differs, but only close to  
the end with a single line.

diff --git a/tmp/compilation-2.16.2 b/tmp/compilation-2.17.14
index 9343e2d..a5e5180 100644
--- a/tmp/compilation-2.16.2
+++ b/tmp/compilation-2.17.14
@@ -1,13 +1,13 @@
 -*- mode: compilation; default-directory: "/tmp/" -*-
-Compilation started at Thu Oct 24 08:55:40
+Compilation started at Thu Oct 24 08:57:18

-/usr/bin/lilypond test.ly
-GNU LilyPond 2.16.2
+/usr/local/tmp/lilypond/out/bin/lilypond test.ly
+GNU LilyPond 2.17.14
 Processing `test.ly'
 Parsing...
-test.ly:0: warning: no \version statement found, please add
+test.ly:1: warning: no \version statement found, please add

-\version "2.16.2"
+\version "2.17.14"

 for future compatibility
 Interpreting  
music...[8][16][24][32][40][48][56][64][72][80][88][96][104][112][
@@ -94,7 +94,6 @@ Drawing systems... (-15.050006 . -15.050006)
 (-15.050006 . -15.050006)
 (-4.34744566929134 . -4.34744566929134)
 (-15.050006 . -15.050006)
-(-4.34744566929134 . -4.34744566929134)
 (-15.050006 . -15.050006)
 (-4.34744566929134 . -4.34744566929134)
 (-15.050006 . -15.050006)
@@ -108,4 +107,4 @@ Layout output to `test.ps'...
 Converting to `./test.pdf'...
 Success: compilation successfully completed

-Compilation finished at Thu Oct 24 08:56:19
+Compilation finished at Thu Oct 24 09:17:09

That single line difference does not make much sense to me.

But what is much more staggering is that 2.16.2 runs 40 seconds here while  
2.17.14 runs 20 minutes (mostly in the "Preprocessing graphical objects"  
stage, namely before line-breaking).

Talk about a performance regression.  I'll have to check whether this  
regression persists into current master, though, but it sure looks ugly.

--
You received this message because this project is configured to send all  
issue notifications to this address.
You may adjust your notification preferences at:
https://code.google.com/hosting/settings