There is a small bug in the
vecLimitDeviationAngleUtility used by
limitMaxDeviationAngle and limitMinDeviationAngle (all
in Vec3 class). Originally the utility was an inline
function meant to be expanded into the two api
routines, with a const arg "dissolving" an IF inside
the utility. In the process of moving old C style code
into the Vec3 class it looks like the ...Utility
function lost its inline-ness while the other two
functions are (probably spuriously) labeled "inline".
When fixing this note that the bool insideOrOutside
parameter name is too ambiguous: does true mean inside
or outside? Either make it "bool insideCone" or use an
enum for the two cases.
(Craig note: see NGC for a slight reworking of these
functions.)
