Hi Gerrit,
Gerrit Voss wrote:
> On Fri, 2008-05-30 at 19:12 +0800, Gerrit Voss wrote:
>> On Wed, 2008-05-28 at 15:45 -0500, [EMAIL PROTECTED] wrote:
>>> Author: cneumann
>>> Date: Wed May 28 15:45:12 2008
>>> New Revision: 1253
>>> Trac changeset: http://opensg.vrsource.org/trac/changeset/1253
>>> ViewVC:
>>> http://realityforge.vrsource.org/viewvc/opensg?view=rev&revision=1253
>>>
>>> Modified:
>>> changed: reduced the number of matrix * {pnt, vec} multiplication functions
>>> and shortened their names.
>> what I don't understand is you took out some functions just to put them
>> back in as free functions. So why change these at all, that doesn't seem
>> to make sense to me.
neither to me ;) ;)
> arrrggg, I missed that these weren't present to begin with, sorry,
> but the general comment below still applies ;-)
ok.
>> On the operator side if the lhs of an operator is one of our classes it
>> should be inside this class. Cluttering the global osg namespace is
>> quite annoying as all the possible variants are listed in case of
>> errors. So you get totally unrelated suggestions as to which possible
>> operators are available. I'm also not sure where they will show up in
>> the documentation.
>>
>> So can we put these operators back into the classes they belong.
>>
>> I only see exceptions if the lhs is not of our type or if you get
>> order conflicts, like having
>>
>> operator *(Pnt &rhs) inside Matrix and operator *(Matrix &rhs) inside
>> point. If resolving this results in ugly include orders free operators
>> can be used as the last resort.
ok, I'll change that.
Carsten
-------------------------------------------------------------------------
This SF.net email is sponsored by: Microsoft
Defy all challenges. Microsoft(R) Visual Studio 2008.
http://clk.atdmt.com/MRT/go/vse0120000070mrt/direct/01/
_______________________________________________
Opensg-core mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/opensg-core
