Carsten Neumann wrote:
> Hi Dirk,
>
> Dirk Reiners wrote:
>> Carsten Neumann wrote:
>>> On the other hand that is also somewhat true for Real128 so we could
>>> keep them in and start worrying about how to handle them on "exotic"
>>> architectures when someone wants to run OpenSG on it. If you think it
>>> would be cleaner to rip them out for now and maybe add them again when
>>> there is sufficient demand, I can prepare a patch to that effect as well.
>> At this point I don't think we need them. I added them for completeness'
>> sake, but I don't htink anybody has ever used them, so if they make
>> trouble, let's get them out for now and worry about it once it's necessary.
>
> Removing Real128 turns out to be slightly more complicated than
> initially hoped for ;)
> I've run into a problem in TypedGeoVectorProperty and its base class
> GeoVectorProperty, which has the following:
> typedef Vec4ld MaxTypeT; (Vec4ld holds Real128 entries).
> which I can not simply change to the obvious Vec4d, because then
>
> // This is the fallback, it has to be implemented by the concrete Props!
> virtual void getValue( MaxTypeT &val, const UInt32 index) const =0;
> virtual void setValue(const MaxTypeT &val, const UInt32 index) =0;
How about renaming these to get/setGenericValue() ?
IMHO this solves more than the overload problem I encounter, it also
indicates that this is not the most efficient interface to use. At the
same time this becomes a little faster, as MaxType can be a "smaller" type:
IntegralProperties: UInt64 -> UInt32
VectorProperties: Vec4ld -> Vec4d
Carsten
-------------------------------------------------------------------------
Take Surveys. Earn Cash. Influence the Future of IT
Join SourceForge.net's Techsay panel and you'll get the chance to share your
opinions on IT & business topics through brief surveys -- and earn cash
http://www.techsay.com/default.php?page=join.php&p=sourceforge&CID=DEVDEV
_______________________________________________
Opensg-core mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/opensg-core
