Cleanup: const args for curve handle calculation
authorCampbell Barton <ideasman42@gmail.com>
Thu, 25 Feb 2016 00:02:46 +0000 (11:02 +1100)
committerCampbell Barton <ideasman42@gmail.com>
Thu, 25 Feb 2016 00:02:46 +0000 (11:02 +1100)
source/blender/blenkernel/intern/colortools.c
source/blender/blenkernel/intern/curve.c

index 32fb6af01b9804f00d16f2aa512d305d24b5c121..bac59c8c62d071854ca1769eb614fd0dd5a67139 100644 (file)
@@ -412,13 +412,16 @@ void curvemap_sethandle(CurveMap *cuma, int type)
 /**
  * reduced copy of #calchandleNurb_intern code in curve.c
  */
-static void calchandle_curvemap(BezTriple *bezt, BezTriple *prev, BezTriple *next, int UNUSED(mode))
+static void calchandle_curvemap(
+        BezTriple *bezt, const BezTriple *prev, const BezTriple *next)
 {
        /* defines to avoid confusion */
 #define p2_h1 ((p2) - 3)
 #define p2_h2 ((p2) + 3)
 
-       float *p1, *p2, *p3, pt[3];
+       const float *p1, *p3;
+       float *p2;
+       float pt[3];
        float len, len_a, len_b;
        float dvec_a[2], dvec_b[2];
 
@@ -546,13 +549,11 @@ static void curvemap_make_table(CurveMap *cuma, const rctf *clipr)
                        bezt[a].h1 = bezt[a].h2 = HD_AUTO;
        }
        
+       const BezTriple *bezt_prev = NULL;
        for (a = 0; a < cuma->totpoint; a++) {
-               if (a == 0)
-                       calchandle_curvemap(bezt, NULL, bezt + 1, 0);
-               else if (a == cuma->totpoint - 1)
-                       calchandle_curvemap(bezt + a, bezt + a - 1, NULL, 0);
-               else
-                       calchandle_curvemap(bezt + a, bezt + a - 1, bezt + a + 1, 0);
+               const BezTriple *bezt_next = (a != cuma->totpoint - 1) ? &bezt[a + 1] : NULL;
+               calchandle_curvemap(&bezt[a], bezt_prev, bezt_next);
+               bezt_prev = &bezt[a];
        }
        
        /* first and last handle need correction, instead of pointing to center of next/prev, 
index 79199f86fa92f4e0691635c063f0a1484b40ce41..3329d3a25ee2be5906384f1c501ea084baf97ee3 100644 (file)
@@ -3120,14 +3120,17 @@ void BKE_curve_bevelList_make(Object *ob, ListBase *nurbs, bool for_render)
 
 /* ****************** HANDLES ************** */
 
-static void calchandleNurb_intern(BezTriple *bezt, BezTriple *prev, BezTriple *next,
-                                  bool is_fcurve, bool skip_align)
+static void calchandleNurb_intern(
+        BezTriple *bezt, const BezTriple *prev, const BezTriple *next,
+        bool is_fcurve, bool skip_align)
 {
        /* defines to avoid confusion */
 #define p2_h1 ((p2) - 3)
 #define p2_h2 ((p2) + 3)
 
-       float *p1, *p2, *p3, pt[3];
+       const float *p1, *p3;
+       float *p2;
+       float pt[3];
        float dvec_a[3], dvec_b[3];
        float len, len_a, len_b;
        float len_ratio;