From 88fe8962434b7fe00fa713a84c58d2dbee481e3b Mon Sep 17 00:00:00 2001 From: Jason Wilkins Date: Wed, 22 Oct 2014 20:03:25 -0500 Subject: Checked each of my (jwilkins) XXX notes. The ones in extern/glew-es have been changed to NOTE instead of XXX GHOST_ContextEGL.cpp: It really does seem that it is not possible to query the swap interval using EGL GHOST_WidnowCocoa.h: The comment referring to Carbon is clearly out of date, so I removed it. math_geom.c: The node about not using tmax again is correct, but the code is kept for a future maintainer who will need to know how to compute it if they modify that code. paint_image_proj.c (2698): The question about integer truncation does not appear to have been resolved. It still seems to be an incorrectly implementation of rounding (I'd suggest using the round function instead of this hack). --- source/blender/blenlib/intern/math_geom.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) (limited to 'source/blender/blenlib') diff --git a/source/blender/blenlib/intern/math_geom.c b/source/blender/blenlib/intern/math_geom.c index da0855ad022..015313431cb 100644 --- a/source/blender/blenlib/intern/math_geom.c +++ b/source/blender/blenlib/intern/math_geom.c @@ -1756,8 +1756,8 @@ bool isect_ray_aabb(const IsectRayAABBData *data, const float bb_min[3], if (tzmin > tmin) tmin = tzmin; - /* XXX jwilkins: tmax does not need to be updated since we don't use it - * keeping this here for future reference */ + /* Note: tmax does not need to be updated since we don't use it + * keeping this here for future reference - jwilkins */ //if (tzmax < tmax) tmax = tzmax; if (tmin_out) -- cgit v1.2.3