> > > One thing that I noticed when working on this patch is that Spline.list > returns a reference to the spline interpolation points (this is actually > noted in the documentation), allowing one again to change the interpolation > points without forcing the spline to be recomputed. This issue was brought > up in #12036 when doctest coverage was addressed, and I've opened a ticket > about this at > > http://trac.sagemath.org/sage_trac/ticket/13530 > > My question is, can I just change Spline.list() to return a copy of the > list of interpolation points, rather than a reference? Strictly speaking, > this would alter the interface and so a DepreciationWarning of some sort > would be appropriate but I can't imagine that anybody is relying on this > (buggy) behavior. Still, maybe there are other reasons for not making this > change -- please let me know if this is the case. > > This seems reasonable, in this case. Changing wrong behavior is called fixing a bug and doesn't need deprecation. That said, I'd be interested in knowing if other software *does* allow this sort of changing things, in which case it could be the case that people would be using it actively and it would need deprecation.
- kcrisman -- You received this message because you are subscribed to the Google Groups "sage-devel" group. To post to this group, send email to [email protected]. To unsubscribe from this group, send email to [email protected]. Visit this group at http://groups.google.com/group/sage-devel?hl=en.
