|
From: | Auto mailings of changes to Lily Issues via Testlilyissues-auto |
Subject: | [Lilypond-auto] [LilyIssues-auto] [testlilyissues:issues] Re: #5513 Avoid some crashes for bad "control-points" property |
Date: | Fri, 26 Apr 2019 10:24:19 -0000 |
"Thomas Morley" address@hidden writes:
There's a recent discussion on -devel about how to draw curves with
more than 4 control-points.
http://lilypond.1069038.n5.nabble.com/stack-smashing-detected-tt221627.html
Not sure yet what to do and how ..., but I'd love to keep the
possibility of entering more than the default amount of control-point
without a type-checking warning here.
Ofcourse ly:slur::print, etc needs 4 control-points not more or less.
Thus a warning while applying ly:slur::print to a not suitable amount
of control-points could warn. (More control-points will need a
different or extended procedure.)
Is this doable with a reasonable amount of work?
I don't think it's worth the trouble: there are so many other ways to
provide faulty data. This issue is about avoiding clobbering the stack,
not more, not less. One could let it bomb out with some type error
instead by being less accepting, but adding additional code/warnings
seems to be mostly useless since most other ways to mess up would not be
covered anyway.
--
David Kastrup
[issues:#5513] Avoid some crashes for bad "control-points" property
Status: Started
Created: Thu Apr 25, 2019 05:35 PM UTC by David Kastrup
Last Updated: Fri Apr 26, 2019 08:45 AM UTC
Owner: David Kastrup
Avoid some crashes for bad "control-points" property
http://codereview.appspot.com/576610043
Sent from sourceforge.net because address@hidden 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.
_______________________________________________ Testlilyissues-auto mailing list address@hidden https://lists.sourceforge.net/lists/listinfo/testlilyissues-auto
[Prev in Thread] | Current Thread | [Next in Thread] |