#7232: fix tachyon segfault introduced by #6542
-------------------------+-------------------------------------------------
Reporter: was | Owner: was
Type: | Status: needs_review
defect | Milestone: sage-6.4
Priority: major | Resolution:
Component: | Merged in:
graphics | Reviewers:
Keywords: | Work issues:
tachyon | Commit:
Authors: | f8cb9754f8ac01164e98fc11fa731a0b66ab0648
Frédéric Chapoton | Stopgaps:
Report Upstream: N/A |
Branch: |
u/chapoton/7232 |
Dependencies: |
-------------------------+-------------------------------------------------
Comment (by niles):
Replying to [comment:20 kcrisman]:
> I think this is a reasonably way to handle this given how long it's
been, and a good doctest, but a brief reminder here about what the
''default'' camera center and look at are might be helpful.
Actually, I think I like the error message better the way it is. Listing
the defaults will be hard to maintain (in the unlikely case that they
change later), unless we print them from some attribute. But that makes
printing the error depend on successfully determining the value of the
attribute, making it slower and more prone to error itself. I think the
error message currently gives all the necessary information with a minimum
of overhead and complication.
Maybe as a compromise, if necessary, print "see init method for default
values" ?
--
Ticket URL: <http://trac.sagemath.org/ticket/7232#comment:21>
Sage <http://www.sagemath.org>
Sage: Creating a Viable Open Source Alternative to Magma, Maple, Mathematica,
and MATLAB
--
You received this message because you are subscribed to the Google Groups
"sage-trac" group.
To unsubscribe from this group and stop receiving emails from it, send an email
to [email protected].
To post to this group, send email to [email protected].
Visit this group at http://groups.google.com/group/sage-trac.
For more options, visit https://groups.google.com/d/optout.
