[Lilypond-auto] [LilyIssues-auto] [testlilyissues:issues] Re: #5345 Crash with specific combination of pitch and omitted note head
Well, I haven't started serious analysis yet, but my hunch is that an array
basically containing {a, b} is iteratively changed into {a, b, a}, {a, b, a,
a}, {a, b, a, a, a} or similar. Basically it is extended all the while and the
iteration never finishes.
---
** [issues:#5345] Crash with specific combination of pitch and omitted note
head**
**Status:** Accepted
**Created:** Fri Jun 15, 2018 12:11 AM UTC by Simon Albrecht
**Last Updated:** Fri Jun 15, 2018 09:33 AM UTC
**Owner:** nobody
Reported by Lukas-Fabian Moser [in this
thread](http://lists.gnu.org/archive/html/lilypond-user/2018-06/msg00222.html)
with the following minimal example:
WARNING: Only test with appropriate configuration of ulimit, see the e-mail
exchange.
:::TeX
\version "2.19.80"
\new Staff <<
{
\clef bass
e,4
}
{
\omit NoteHead
e,4
}
>>
Lukas says:
> Everything here (except the \version) seems to be necessary. I can reproduce
> the crash with some other configurations as well (another pitch in tenor
> clef, for instance), but removing any single command seems to resolve the
> problem. Also, putting each "voice" into a regular \new Voice (as in Harm's
> proposal) makes the problem disappear. Just as taking f, instead of e, for
> the pitch does (which could point to ledger lines).
David K. says the problem appears to be in the page builder.
---
Sent from sourceforge.net because testlilyissues-a...@lists.sourceforge.net is
subscribed to https://sourceforge.net/p/testlilyissues/issues/
To unsubscribe from further messages, a project admin can change settings at
https://sourceforge.net/p/testlilyissues/admin/issues/options. Or, if this is
a mailing list, you can unsubscribe from the mailing list.--
Check out the vibrant tech community on one of the world's most
engaging tech sites, Slashdot.org! http://sdm.link/slashdot___
Testlilyissues-auto mailing list
testlilyissues-a...@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/testlilyissues-auto
[Lilypond-auto] [LilyIssues-auto] [testlilyissues:issues] Re: #5345 Crash with specific combination of pitch and omitted note head
Looks to me like a problem with a missing pure-impure-container -- checking the
extent has a side effect, so you create a new layout, and off you go. But I
haven't checked out the exact code.
---
** [issues:#5345] Crash with specific combination of pitch and omitted note
head**
**Status:** Accepted
**Created:** Fri Jun 15, 2018 12:11 AM UTC by Simon Albrecht
**Last Updated:** Fri Jun 15, 2018 09:33 AM UTC
**Owner:** nobody
Reported by Lukas-Fabian Moser [in this
thread](http://lists.gnu.org/archive/html/lilypond-user/2018-06/msg00222.html)
with the following minimal example:
WARNING: Only test with appropriate configuration of ulimit, see the e-mail
exchange.
:::TeX
\version "2.19.80"
\new Staff <<
{
\clef bass
e,4
}
{
\omit NoteHead
e,4
}
>>
Lukas says:
> Everything here (except the \version) seems to be necessary. I can reproduce
> the crash with some other configurations as well (another pitch in tenor
> clef, for instance), but removing any single command seems to resolve the
> problem. Also, putting each "voice" into a regular \new Voice (as in Harm's
> proposal) makes the problem disappear. Just as taking f, instead of e, for
> the pitch does (which could point to ledger lines).
David K. says the problem appears to be in the page builder.
---
Sent from sourceforge.net because testlilyissues-a...@lists.sourceforge.net is
subscribed to https://sourceforge.net/p/testlilyissues/issues/
To unsubscribe from further messages, a project admin can change settings at
https://sourceforge.net/p/testlilyissues/admin/issues/options. Or, if this is
a mailing list, you can unsubscribe from the mailing list.--
Check out the vibrant tech community on one of the world's most
engaging tech sites, Slashdot.org! http://sdm.link/slashdot___
Testlilyissues-auto mailing list
testlilyissues-a...@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/testlilyissues-auto
[Lilypond-auto] [LilyIssues-auto] [testlilyissues:issues] Re: #5345 Crash with specific combination of pitch and omitted note head
Well, the patch contains this part:
+ + // When the extents of two
ledgers at the same
+ // vertical position overlap horizontally, we merge
+ // them together to produce a single stencil. In rare
+ // cases they do not overlap and we do not merge them.
+
if (lr.ledger_extents_.find (lpos) == lr.ledger_extents_.end
())
-lr.ledger_extents_[lpos] = x_extent;
+// Found nothing for this lpos.
+lr.ledger_extents_[lpos].push_back(x_extent);
else
-lr.ledger_extents_[lpos].unite (x_extent);
I suspect that "do not merge" here translates to "eat all memory and crash".
But I'll have to do more of an analysis to be sure.
---
** [issues:#5345] Crash with specific combination of pitch and omitted note
head**
**Status:** Accepted
**Created:** Fri Jun 15, 2018 12:11 AM UTC by Simon Albrecht
**Last Updated:** Fri Jun 15, 2018 09:33 AM UTC
**Owner:** nobody
Reported by Lukas-Fabian Moser [in this
thread](http://lists.gnu.org/archive/html/lilypond-user/2018-06/msg00222.html)
with the following minimal example:
WARNING: Only test with appropriate configuration of ulimit, see the e-mail
exchange.
:::TeX
\version "2.19.80"
\new Staff <<
{
\clef bass
e,4
}
{
\omit NoteHead
e,4
}
>>
Lukas says:
> Everything here (except the \version) seems to be necessary. I can reproduce
> the crash with some other configurations as well (another pitch in tenor
> clef, for instance), but removing any single command seems to resolve the
> problem. Also, putting each "voice" into a regular \new Voice (as in Harm's
> proposal) makes the problem disappear. Just as taking f, instead of e, for
> the pitch does (which could point to ledger lines).
David K. says the problem appears to be in the page builder.
---
Sent from sourceforge.net because testlilyissues-a...@lists.sourceforge.net is
subscribed to https://sourceforge.net/p/testlilyissues/issues/
To unsubscribe from further messages, a project admin can change settings at
https://sourceforge.net/p/testlilyissues/admin/issues/options. Or, if this is
a mailing list, you can unsubscribe from the mailing list.--
Check out the vibrant tech community on one of the world's most
engaging tech sites, Slashdot.org! http://sdm.link/slashdot___
Testlilyissues-auto mailing list
testlilyissues-a...@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/testlilyissues-auto
[Lilypond-auto] [LilyIssues-auto] [testlilyissues:issues] Re: #5345 Crash with specific combination of pitch and omitted note head
Oh wow. I did not think of bijecting, and "don't merge non-overkapping ledger
lines" is a theme that might be related to omitted note heads. The pitch would
require a ledger line as noted by Lukas, and its double occurence would warrant
some sort of merging.
This helps a lot.
---
** [issues:#5345] Crash with specific combination of pitch and omitted note
head**
**Status:** Accepted
**Created:** Fri Jun 15, 2018 12:11 AM UTC by Simon Albrecht
**Last Updated:** Fri Jun 15, 2018 09:33 AM UTC
**Owner:** nobody
Reported by Lukas-Fabian Moser [in this
thread](http://lists.gnu.org/archive/html/lilypond-user/2018-06/msg00222.html)
with the following minimal example:
WARNING: Only test with appropriate configuration of ulimit, see the e-mail
exchange.
:::TeX
\version "2.19.80"
\new Staff <<
{
\clef bass
e,4
}
{
\omit NoteHead
e,4
}
>>
Lukas says:
> Everything here (except the \version) seems to be necessary. I can reproduce
> the crash with some other configurations as well (another pitch in tenor
> clef, for instance), but removing any single command seems to resolve the
> problem. Also, putting each "voice" into a regular \new Voice (as in Harm's
> proposal) makes the problem disappear. Just as taking f, instead of e, for
> the pitch does (which could point to ledger lines).
David K. says the problem appears to be in the page builder.
---
Sent from sourceforge.net because testlilyissues-a...@lists.sourceforge.net is
subscribed to https://sourceforge.net/p/testlilyissues/issues/
To unsubscribe from further messages, a project admin can change settings at
https://sourceforge.net/p/testlilyissues/admin/issues/options. Or, if this is
a mailing list, you can unsubscribe from the mailing list.--
Check out the vibrant tech community on one of the world's most
engaging tech sites, Slashdot.org! http://sdm.link/slashdot___
Testlilyissues-auto mailing list
testlilyissues-a...@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/testlilyissues-auto
