[ODE] Making it build again

[Jon Watte]

The double& in the collision check would break compile on any 
platform with dReal == float.

I wouldn't say "bothered by warnings." It's well-accepted practice 
in the industry that WARNINGS ARE ERRORS. If you're not treating 
warnings as errors, well, you should! (I don't want to argue about 
it, but you can't find anyone who's any good who wouldn't agree).

ODE was warning free before the check-ins, and should remain so, by 
fixing the code that it warns about. It's the only way to keep the 
system sane.

Cheers,

			/ h+


-----Original Message-----
From: Jeffrey Smith [mailto:jeffreys at Softimage.com]
Sent: Friday, September 24, 2004 9:02 AM
To: Jon Watte; Ode at Q12. Org
Subject: RE: [ODE] Making it build again


> I updated CVS today (haven't done so in a while) and found that 
> some things didn't build cleanly (or at all) like they used to. 

It may have been me. Although I verified that ODE compiled on Windows
and Linux before I checked it, I on;y tested with dReal == double.  
I'll recheck with dReal==float and make any necessary fixes.

The problems you list are minor; is there anything which actually
doesn't compile?  Or are you just being bothered by warnings?

-Jeff