[Top][All Lists]

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

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

From: Edward Rosten
Subject: Re: [Toon-members] Bug report: TooN - missing function (fwd)
Date: Thu, 17 May 2007 10:44:14 -0600 (MDT)

On Thu, 17 May 2007, Tom Drummond wrote:


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.


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


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.


Gerhard Reitmayr
MIL, Engineering Department, Cambridge University

Toon-members mailing list

reply via email to

[Prev in Thread] Current Thread [Next in Thread]