Re: [Lilypond-auto] Issue 3255 in lilypond: \with-dimensions doesn't affect the overall size of a TextScript

[lilypond]

Updates:
        Labels: -Patch-new Patch-needs_work

Comment #80 on issue 3255 by address@hidden: \with-dimensions doesn't  
affect the overall size of a TextScript
http://code.google.com/p/lilypond/issues/detail?id=3255

Fails make check on a couple of .ly files (it's tricky to hunt them out  
when running multi-cpu however...).

One example:

--snip--
% ****************************************************************
% ly snippet:
% ****************************************************************
\sourcefilename 
"/tmp/lilypond-autobuild/input/regression/harp-pedals-sanity-checks.ly"
\sourcefileline 0
\version "2.17.6"

#(ly:set-option 'warning-as-error #f)
#(ly:expect-warning (_ "Harp pedal diagram contains dividers at positions  
~a.  Normally, there is only one divide
r after the third pedal.") '(1 3 5))
#(ly:expect-warning (_ "Harp pedal diagram contains dividers at positions  
~a.  Normally, there is only one divide
r after the third pedal.") '(4))
#(ly:expect-warning (_ "Harp pedal diagram contains ~a pedals rather than  
the usual 7.") 5)
#(ly:expect-warning (_ "Harp pedal diagram does not contain a divider  
(usually after third pedal)."))


\header {
  texidoc = "The harp-pedal markup function does some sanity checks. All
the diagrams here violate the standard (7 pedals with divider after third),  
so
a warning is printed out, but they should still look okay."
}

\relative c'' {
  \override Score.PaperColumn.keep-inside-line = ##f
  % Sanity checks: #pedals != 7:
  c1^\markup \harp-pedal #"^-v|--"
  % Sanity checks: no divider, multiple dividers, divider on wrong position:
  c1^\markup \harp-pedal #"^-v--v^"
  c1^\markup \harp-pedal #"^|-v|--|v^"
  c1^\markup \harp-pedal #"^-v-|-v^"
}



% ****************************************************************
% end ly snippet
% ****************************************************************
--snip--

the error is

--snip--
Processing  
`/tmp/build-lilypond-autobuild/out/lybook-testdb/86/lily-6c3e40f2.ly'
Parsing...
Renaming input to:  
`/tmp/lilypond-autobuild/input/regression/harp-pedals-sanity-checks.ly'
Interpreting music...
Preprocessing graphical objects...Backtrace:
In unknown file:
   ?:  5* [#<procedure #f (x)> "86/lily-6c3e40f2.ly"]
In /tmp/build-lilypond-autobuild/out/share/lilypond/current/scm/lily.scm:
 987:  6* (let* (# # #) (if separate-logs #) (if ping-log #) ...)
 998:  7* [lilypond-file #<procedure #f (key  
failed-file)> "86/lily-6c3e40f2.ly"]
1033:  8  [catch ly-file-failed #<procedure #f ()> #<procedure #f (x .  
args)>]
In unknown file:
   ?:  9* [#<procedure #f ()>]
In /tmp/build-lilypond-autobuild/out/share/lilypond/current/scm/lily.scm:
1034: 10* [ly:parse-file "86/lily-6c3e40f2.ly"]
In /tmp/build-lilypond-autobuild/out/share/lilypond/current/ly/init.ly:
  53: 11* (let* ((book-handler #)) (cond (# #) (# #)))
  56: 12  (cond (# #) (# #))
In  
/tmp/build-lilypond-autobuild/out/share/lilypond/current/scm/lily-library.scm:
    ...
 242: 13  [ly:book-process-to-systems #<Book> #< Output_def> ...]
In unknown file:
   ?: 14* [ly:spacing-spanner::set-springs #<Grob SpacingSpanner >]
   ?: 15* [ly:separation-item::calc-skylines #<Grob NonMusicalPaperColumn >]
   ?: 16* [ly:hara-kiri-group-spanner::pure-height #<Grob VerticalAxisGroup  
> 0 ...]
   ?: 17* [ly:axis-group-interface::adjacent-pure-heights #<Grob  
VerticalAxisGroup >]
   ?: 18* [#<unpure-pure-call 40c497a8> #<Grob TextScript > 6 8]
   ?: 19* [ly:grob::stencil-height #<Grob TextScript >]
   ?: 20* [ly:text-interface::print #<Grob TextScript >]
   ?: 21* [harp-pedal-markup #< Output_def> (# # #) "^-v-|-v^"]
   ?: 22* (let (# # #) (let* # #))
In  
/tmp/build-lilypond-autobuild/out/share/lilypond/current/scm/harp-pedals.scm:
  67: 23  (let* (# # # # ...) (skyline-dimensions-stencil visible-stencil #  
#))
 133: 24  (skyline-dimensions-stencil visible-stencil # #)

/tmp/build-lilypond-autobuild/out/share/lilypond/current/scm/harp-pedals.scm:133:5:  
In  
expression (skyline-dimensions-stencil visible-stencil  
(ly:stencil-extent visible-stencil X) ...):
/tmp/build-lilypond-autobuild/out/share/lilypond/current/scm/harp-pedals.scm:133:5:  
Unbound  
variable: skyline-dimensions-stencil
--snip--

I think.


--
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