[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Lilypond-auto] Issue 3663 in lilypond: Crash with \repeat ... \alte
From: |
lilypond |
Subject: |
Re: [Lilypond-auto] Issue 3663 in lilypond: Crash with \repeat ... \alternative and \remove "Bar_engraver" |
Date: |
Fri, 22 Nov 2013 13:24:25 +0000 |
Updates:
Labels: -Patch-push Fixed_2_19_0 Backport_2_18_0
Comment #8 on issue 3663 by address@hidden: Crash with \repeat ...
\alternative and \remove "Bar_engraver"
http://code.google.com/p/lilypond/issues/detail?id=3663
I am _not_ happy about this fix. It's sort of "muddle through" when
presented with a case previously causing a crash, and the result is not the
same as in 2.16.
I certainly hope that somebody will eventually sort this out in
bar-line.scm.
Pushed to staging as
commit 539fa974c50b632fd6d3c4a65dbf988bb2880e2a
Author: David Kastrup <address@hidden>
Date: Sun Nov 17 19:29:06 2013 +0100
Issue 3663: Crash with \repeat ... \alternative and
\remove "Bar_engraver"
This approach is just poking around in the dark and removing Scheme
error messages. In particular substituting the original glyph for
left-bar-line and right-bar-line (when missing) in calls to
span-bar::compound-bar-line seems fishy, and the original logic
for setting left-bar-broken if left-bar-line is not even present
is also not clear to me.
The results for the example in the bug report:
form = \new Staff \with {
\remove "Bar_engraver"
} \repeat volta 2 {
s1
} \alternative {
s1 % first ending
s1 % second ending
} % form
\score {
<<
\form
>>
} % score
are also not identical with the 2.16 results: the volta brackets now
run into each other; before there was a gap.
It fixes the crash. I have no idea what a proper fix would look like.
--
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
- [Lilypond-auto] Issue 3663 in lilypond: Crash with \repeat ... \alternative and \remove "Bar_engraver", lilypond, 2013/11/14
- Re: [Lilypond-auto] Issue 3663 in lilypond: Crash with \repeat ... \alternative and \remove "Bar_engraver", lilypond, 2013/11/17
- Re: [Lilypond-auto] Issue 3663 in lilypond: Crash with \repeat ... \alternative and \remove "Bar_engraver", lilypond, 2013/11/17
- Re: [Lilypond-auto] Issue 3663 in lilypond: Crash with \repeat ... \alternative and \remove "Bar_engraver", lilypond, 2013/11/17
- Re: [Lilypond-auto] Issue 3663 in lilypond: Crash with \repeat ... \alternative and \remove "Bar_engraver", lilypond, 2013/11/17
- Re: [Lilypond-auto] Issue 3663 in lilypond: Crash with \repeat ... \alternative and \remove "Bar_engraver", lilypond, 2013/11/19
- Re: [Lilypond-auto] Issue 3663 in lilypond: Crash with \repeat ... \alternative and \remove "Bar_engraver", lilypond, 2013/11/19
- Re: [Lilypond-auto] Issue 3663 in lilypond: Crash with \repeat ... \alternative and \remove "Bar_engraver", lilypond, 2013/11/22
- Re: [Lilypond-auto] Issue 3663 in lilypond: Crash with \repeat ... \alternative and \remove "Bar_engraver",
lilypond <=
- Re: [Lilypond-auto] Issue 3663 in lilypond: Crash with \repeat ... \alternative and \remove "Bar_engraver", lilypond, 2013/11/22
- Re: [Lilypond-auto] Issue 3663 in lilypond: Crash with \repeat ... \alternative and \remove "Bar_engraver", lilypond, 2013/11/24