Cleanup: Rearrange the order of isect_ray_ray_v3 parameters
authormano-wii <germano.costa@ig.com.br>
Tue, 23 Jul 2019 11:11:46 +0000 (08:11 -0300)
committermano-wii <germano.costa@ig.com.br>
Tue, 23 Jul 2019 11:11:46 +0000 (08:11 -0300)
And remove redundant add operation.

source/blender/blenlib/intern/math_geom.c
source/blender/editors/transform/transform_constraints.c

index 8b715ebf87bf8106cc74bbe64cdcb54e0fb980bf..dbb4e0d3859f7b45471f824c46dd10e0d738f011 100644 (file)
@@ -2826,12 +2826,12 @@ bool isect_ray_ray_v3(const float ray_origin_a[3],
   sub_v3_v3v3(c, n, t);
 
   if (r_lambda_a != NULL) {
-    cross_v3_v3v3(cray, c, ray_direction_a);
+    cross_v3_v3v3(cray, c, ray_direction_b);
     *r_lambda_a = dot_v3v3(cray, n) / nlen;
   }
 
   if (r_lambda_b != NULL) {
-    cross_v3_v3v3(cray, c, ray_direction_b);
+    cross_v3_v3v3(cray, c, ray_direction_a);
     *r_lambda_b = dot_v3v3(cray, n) / nlen;
   }
 
index 208242d53b38f90fafd5e9cf6aa46050b42b793f..3a204d0d1578c6e2860b184b7b4ed1adc545eae2 100644 (file)
@@ -263,9 +263,8 @@ static void axisProjection(const TransInfo *t,
       /* Use ray-ray intersection instead of line-line because this gave
        * precision issues adding small values to large numbers. */
       float mul;
-      if (isect_ray_ray_v3(v, norm, t_con_center, axis, &mul, NULL)) {
-        madd_v3_v3v3fl(out, t_con_center, axis, mul);
-        sub_v3_v3(out, t_con_center);
+      if (isect_ray_ray_v3(t_con_center, axis, v, norm, &mul, NULL)) {
+        mul_v3_v3fl(out, axis, mul);
       }
       else {
         /* In practice this should never fail. */