Re: [Toon-members] Bug report: TooN - missing function (fwd)

[Edward Rosten]

On Thu, 17 May 2007, Tom Drummond wrote:

> Ed,
>
> On reflection, I agree.  I think this should be removed from TooN.  We don't 
> have methods for Matrix * double* or double* * double*, so we probably 
> shouldn't have this.

OK, now I'm convinced.

> You might wish to communicate the safe and unsafe ways of doing this to our 
> external user and the reasons we don't have this available as a method.  The 
> fixes take almost no external code and can be done in-line and allow the user 
> to choose safe or not.
>
> Cheers

I've emailed the user. I'll remove it when I get a reply (or in a bit if I 
don't).

-Ed




> On 17 May 2007, at 12:28, Gerhard Reitmayr wrote:
>
> > Edward Rosten wrote:
> > > On Tue, 15 May 2007, Gerhard Reitmayr wrote:
> > >
> > > > I am all for opening up the mailing lists.
> > >
> > > Sounds good. Do you opt for creating a second open list?
> >
> > no, i am ok with just making the current list public. it does not seem
> > that we discuss really secret things here :)
> >
> > > > Unrelated question about the patch:
> > > > what is this method used for?
> > >
> > > I don't know, but it was there.
> > >
> > > > do we want such a not-type-safe API?
> > >
> > > I have naver used it, and I can't see why you would. But it seems that
> > > someone wants to.
> >
> > well, I certainly would thank them for the heads-up and explain that the
> > function should not exist and a cast on the user side is the way resolve
> > it. In my point of view the method invites abuse without overhead (ugly
> > cast to use).
> >
> > I agree that I am probably anal about this, as it definitely does not
> > matter in current code or any future code I will write using TooN. So, I
> > trust that I have made my point with this email :) and I am fine with
> > whatever you decide to do.
> >
> > cheers,
> >  Gerhard
> >
> > --
> > Gerhard Reitmayr
> > MIL, Engineering Department, Cambridge University
> > http://www.eng.cam.ac.uk/~gr281/
> >
> >
> > _______________________________________________
> > Toon-members mailing list
> > address@hidden
> > http://lists.nongnu.org/mailman/listinfo/toon-members