code cleanup: replace most DO_MINMAX2 -> minmax_v2v2_v2
authorCampbell Barton <ideasman42@gmail.com>
Thu, 15 Nov 2012 22:20:18 +0000 (22:20 +0000)
committerCampbell Barton <ideasman42@gmail.com>
Thu, 15 Nov 2012 22:20:18 +0000 (22:20 +0000)
also add UNPACK macros's. handy for printing vectors for eg.

13 files changed:
source/blender/blenkernel/intern/customdata.c
source/blender/blenkernel/intern/mask_evaluate.c
source/blender/blenkernel/intern/tracking.c
source/blender/blenlib/BLI_utildefines.h
source/blender/compositor/operations/COM_KeyingScreenOperation.cpp
source/blender/editors/sculpt_paint/paint_image.c
source/blender/editors/space_clip/clip_draw.c
source/blender/editors/space_clip/clip_editor.c
source/blender/editors/space_node/node_group.c
source/blender/editors/transform/transform_conversions.c
source/blender/editors/uvedit/uvedit_ops.c
source/blender/editors/uvedit/uvedit_parametrizer.c
source/blender/editors/uvedit/uvedit_unwrap_ops.c

index 93c776ae30e2a3a94d33c8191de689f9717b6d1e..b2f8db0dcce2c7fcd52ca9ad62f39d0baa15a7c2 100644 (file)
@@ -761,7 +761,7 @@ static void layerDoMinMax_mloopuv(void *data, void *vmin, void *vmax)
 {
        MLoopUV *min = vmin, *max = vmax, *luv = data;
 
-       DO_MINMAX2(luv->uv, min->uv, max->uv);
+       minmax_v2v2_v2(min->uv, max->uv, luv->uv);
 }
 
 static void layerAdd_mloopuv(void *data1, void *data2)
@@ -833,7 +833,7 @@ static void layerDoMinMax_mloop_origspace(void *data, void *vmin, void *vmax)
 {
        OrigSpaceLoop *min = vmin, *max = vmax, *luv = data;
 
-       DO_MINMAX2(luv->uv, min->uv, max->uv);
+       minmax_v2v2_v2(min->uv, max->uv, luv->uv);
 }
 
 static void layerAdd_mloop_origspace(void *data1, void *data2)
index 7d89678d36f2880d2f05e39baf4d20c39e76900a..a2f6b3c1929c374fb079f0cb9bc827bfab0cbd58 100644 (file)
@@ -289,10 +289,10 @@ static void feather_bucket_check_intersect(float (*feather_points)[2], int tot_f
                        /* collapse loop with smaller AABB */
                        for (k = 0; k < tot_feather_point; k++) {
                                if (k >= check_b && k <= cur_a) {
-                                       DO_MINMAX2(feather_points[k], min_a, max_a);
+                                       minmax_v2v2_v2(min_a, max_a, feather_points[k]);
                                }
                                else {
-                                       DO_MINMAX2(feather_points[k], min_b, max_b);
+                                       minmax_v2v2_v2(min_b, max_b, feather_points[k]);
                                }
                        }
 
@@ -379,7 +379,7 @@ void BKE_mask_spline_feather_collapse_inner_loops(MaskSpline *spline, float (*fe
                int next = i + 1;
                float delta;
 
-               DO_MINMAX2(feather_points[i], min, max);
+               minmax_v2v2_v2(min, max, feather_points[i]);
 
                if (next == tot_feather_point) {
                        if (spline->flag & MASK_SPLINE_CYCLIC)
index b174f27b7e1609378870e1cf2ac05a704ff5d990..30b484010469b29a7d2eba153a750e9889308cb2 100644 (file)
@@ -1124,10 +1124,10 @@ void BKE_tracking_marker_pattern_minmax(const MovieTrackingMarker *marker, float
 {
        INIT_MINMAX2(min, max);
 
-       DO_MINMAX2(marker->pattern_corners[0], min, max);
-       DO_MINMAX2(marker->pattern_corners[1], min, max);
-       DO_MINMAX2(marker->pattern_corners[2], min, max);
-       DO_MINMAX2(marker->pattern_corners[3], min, max);
+       minmax_v2v2_v2(min, max, marker->pattern_corners[0]);
+       minmax_v2v2_v2(min, max, marker->pattern_corners[1]);
+       minmax_v2v2_v2(min, max, marker->pattern_corners[2]);
+       minmax_v2v2_v2(min, max, marker->pattern_corners[3]);
 }
 
 void BKE_tracking_marker_get_subframe_position(MovieTrackingTrack *track, float framenr, float pos[2])
@@ -3161,7 +3161,7 @@ static int stabilization_median_point_get(MovieTracking *tracking, int framenr,
                if (track->flag & TRACK_USE_2D_STAB) {
                        MovieTrackingMarker *marker = BKE_tracking_marker_get(track, framenr);
 
-                       DO_MINMAX2(marker->pos, min, max);
+                       minmax_v2v2_v2(min, max, marker->pos);
 
                        ok = TRUE;
                }
index 056fa916da11d367168cb532ba1a3504c86d79bd..a22aa0c13d7ec4e5a87327a369e192207c1fad92 100644 (file)
 #define IN_RANGE(a, b, c) ((b < c) ? ((b < a && a < c) ? 1 : 0) : ((c < a && a < b) ? 1 : 0))
 #define IN_RANGE_INCL(a, b, c) ((b < c) ? ((b <= a && a <= c) ? 1 : 0) : ((c <= a && a <= b) ? 1 : 0))
 
+/* unpack vector for args */
+#define UNPACK2(a)  ((a)[0]), ((a)[1])
+#define UNPACK3(a)  ((a)[0]), ((a)[1]), ((a)[2])
+#define UNPACK4(a)  ((a)[0]), ((a)[1]), ((a)[2]), ((a)[3])
+/* op may be '&' or '*' */
+#define UNPACK2OP(a, op)  op((a)[0]), op((a)[1])
+#define UNPACK3OP(a, op)  op((a)[0]), op((a)[1]), op((a)[2])
+#define UNPACK4OP(a, op)  op((a)[0]), op((a)[1]), op((a)[2]), op((a)[3])
+
 /* array helpers */
 #define ARRAY_LAST_ITEM(arr_start, arr_dtype, elem_size, tot)                 \
        (arr_dtype *)((char *)arr_start + (elem_size * (tot - 1)))
index 0874e2f59be3c578ba88d490ec34a4bebf86a230..201dc99eb9e2d3250c675d6e598b037333257c3c 100644 (file)
@@ -202,9 +202,9 @@ KeyingScreenOperation::TriangulationData *KeyingScreenOperation::buildVoronoiTri
 
                        INIT_MINMAX2(min, max);
 
-                       DO_MINMAX2(a->co, min, max);
-                       DO_MINMAX2(b->co, min, max);
-                       DO_MINMAX2(c->co, min, max);
+                       minmax_v2v2_v2(min, max, a->co);
+                       minmax_v2v2_v2(min, max, b->co);
+                       minmax_v2v2_v2(min, max, c->co);
 
                        rect->xmin = min[0];
                        rect->ymin = min[1];
index 192ad35109a1648321370cf084247896e1011cc2..676f033af3210696aa06615ee93c5445c4e794c0 100644 (file)
@@ -1059,11 +1059,11 @@ static int pixel_bounds_uv(
        
        INIT_MINMAX2(min_uv, max_uv);
        
-       DO_MINMAX2(uv1, min_uv, max_uv);
-       DO_MINMAX2(uv2, min_uv, max_uv);
-       DO_MINMAX2(uv3, min_uv, max_uv);
+       minmax_v2v2_v2(min_uv, max_uv, uv1);
+       minmax_v2v2_v2(min_uv, max_uv, uv2);
+       minmax_v2v2_v2(min_uv, max_uv, uv3);
        if (is_quad)
-               DO_MINMAX2(uv4, min_uv, max_uv);
+               minmax_v2v2_v2(min_uv, max_uv, uv4);
        
        bounds_px->xmin = (int)(ibuf_x * min_uv[0]);
        bounds_px->ymin = (int)(ibuf_y * min_uv[1]);
@@ -1089,7 +1089,7 @@ static int pixel_bounds_array(float (*uv)[2], rcti *bounds_px, const int ibuf_x,
        INIT_MINMAX2(min_uv, max_uv);
        
        while (tot--) {
-               DO_MINMAX2((*uv), min_uv, max_uv);
+               minmax_v2v2_v2(min_uv, max_uv, (*uv));
                uv++;
        }
        
@@ -2940,7 +2940,7 @@ static void project_paint_delayed_face_init(ProjPaintState *ps, const MFace *mf,
        fidx = mf->v4 ? 3 : 2;
        do {
                vCoSS = ps->screenCoords[*(&mf->v1 + fidx)];
-               DO_MINMAX2(vCoSS, min, max);
+               minmax_v2v2_v2(min, max, vCoSS);
        } while (fidx--);
        
        project_paint_bucket_bounds(ps, min, max, bucketMin, bucketMax);
@@ -3205,7 +3205,7 @@ static void project_paint_begin(ProjPaintState *ps)
                        /* screen space, not clamped */
                        projScreenCo[0] = (float)(ps->winx / 2.0f) + (ps->winx / 2.0f) * projScreenCo[0];
                        projScreenCo[1] = (float)(ps->winy / 2.0f) + (ps->winy / 2.0f) * projScreenCo[1];
-                       DO_MINMAX2(projScreenCo, ps->screenMin, ps->screenMax);
+                       minmax_v2v2_v2(ps->screenMin, ps->screenMax, projScreenCo);
                }
        }
        else {
@@ -3220,7 +3220,7 @@ static void project_paint_begin(ProjPaintState *ps)
                                projScreenCo[0] = (float)(ps->winx / 2.0f) + (ps->winx / 2.0f) * projScreenCo[0] / projScreenCo[3];
                                projScreenCo[1] = (float)(ps->winy / 2.0f) + (ps->winy / 2.0f) * projScreenCo[1] / projScreenCo[3];
                                projScreenCo[2] = projScreenCo[2] / projScreenCo[3]; /* Use the depth for bucket point occlusion */
-                               DO_MINMAX2(projScreenCo, ps->screenMin, ps->screenMax);
+                               minmax_v2v2_v2(ps->screenMin, ps->screenMax, projScreenCo);
                        }
                        else {
                                /* TODO - deal with cases where 1 side of a face goes behind the view ?
index d523d59892528af0ffeef3c263569bc2276cb74e..9cf389c4508cd9ce450363a80fc761d4d8005368 100644 (file)
@@ -1284,7 +1284,7 @@ static void draw_distortion(SpaceClip *sc, ARegion *ar, MovieClip *clip,
 
                        BKE_tracking_undistort_v2(tracking, pos, tpos);
 
-                       DO_MINMAX2(tpos, min, max);
+                       minmax_v2v2_v2(min, max, tpos);
                }
 
                copy_v2_v2(pos, min);
index 71589517c83cf213d4eb01cd8d6497ebc7bd427c..1a62af39600131bb60b448d97ca8cbb05b7ebc57 100644 (file)
@@ -349,7 +349,7 @@ static int selected_boundbox(SpaceClip *sc, float min[2], float max[2])
 
                                mul_v3_m4v3(pos, sc->stabmat, pos);
 
-                               DO_MINMAX2(pos, min, max);
+                               minmax_v2v2_v2(min, max, pos);
 
                                ok = TRUE;
                        }
index 5f8b5db7766238896bb133c144594722c9dd4e3b..4dd9c89375d64fb8a29efd58c1ab1cbe1e37bfc8 100644 (file)
  *  \ingroup spnode
  */
 
+#include <stdlib.h>
+
 #include "MEM_guardedalloc.h"
 
 #include "DNA_node_types.h"
 #include "DNA_object_types.h"
 #include "DNA_anim_types.h"
 
-#include "BLI_blenlib.h"
+#include "BLI_listbase.h"
+#include "BLI_string.h"
+#include "BLI_rect.h"
+#include "BLI_math.h"
 
 #include "BKE_action.h"
 #include "BKE_animsys.h"
@@ -865,7 +870,7 @@ static void node_get_selected_minmax(bNodeTree *ntree, bNode *gnode, float *min,
                if (node == gnode)
                        continue;
                if (node->flag & NODE_SELECT) {
-                       DO_MINMAX2((&node->locx), min, max);
+                       minmax_v2v2_v2(min, max, &node->locx);
                }
        }
 }
index 9bbd3f59cbfed3602394bf37ede4d82edacbe185..5b79b69f7ec5aaaee0bbab5226e566b5baa76e27 100644 (file)
@@ -2486,7 +2486,7 @@ int clipUVTransform(TransInfo *t, float *vec, int resize)
        max[0] = aspx; max[1] = aspy;
 
        for (a = 0, td = t->data; a < t->total; a++, td++) {
-               DO_MINMAX2(td->loc, min, max);
+               minmax_v2v2_v2(min, max, td->loc);
        }
 
        if (resize) {
index a37a1e774542cbf1939db55baf01add9ecefc4b2..6664f8e860ebcb2f8575a6a83f830d4d8e090415 100644 (file)
@@ -616,7 +616,7 @@ int ED_uvedit_minmax(Scene *scene, Image *ima, Object *obedit, float r_min[2], f
                BM_ITER_ELEM (l, &liter, efa, BM_LOOPS_OF_FACE) {
                        if (uvedit_uv_select_test(em, scene, l)) {
                                luv = CustomData_bmesh_get(&em->bm->ldata, l->head.data, CD_MLOOPUV);
-                               DO_MINMAX2(luv->uv, r_min, r_max);
+                               minmax_v2v2_v2(r_min, r_max, luv->uv);
                                sel = 1;
                        }
                }
@@ -1327,7 +1327,7 @@ static void weld_align_uv(bContext *C, int tool)
                        BM_ITER_ELEM (l, &liter, efa, BM_LOOPS_OF_FACE) {
                                if (uvedit_uv_select_test(em, scene, l)) {
                                        MLoopUV *luv = CustomData_bmesh_get(&em->bm->ldata, l->head.data, CD_MLOOPUV);
-                                       DO_MINMAX2(luv->uv, min, max);
+                                       minmax_v2v2_v2(min, max, luv->uv);
                                }
                        }
                }
index 5b6125b558b2bf312a036605691828a945453d1c..8703234122a76cb6c396a896ae6321e7b6f39443 100644 (file)
@@ -450,7 +450,7 @@ static void p_chart_uv_bbox(PChart *chart, float minv[2], float maxv[2])
        INIT_MINMAX2(minv, maxv);
 
        for (v = chart->verts; v; v = v->nextlink) {
-               DO_MINMAX2(v->uv, minv, maxv);
+               minmax_v2v2_v2(minv, maxv, v->uv);
        }
 }
 
@@ -3891,9 +3891,9 @@ static void p_smooth(PChart *chart)
 
                INIT_MINMAX2(fmin, fmax);
 
-               DO_MINMAX2(e1->vert->uv, fmin, fmax);
-               DO_MINMAX2(e2->vert->uv, fmin, fmax);
-               DO_MINMAX2(e3->vert->uv, fmin, fmax);
+               minmax_v2v2_v2(fmin, fmax, e1->vert->uv);
+               minmax_v2v2_v2(fmin, fmax, e2->vert->uv);
+               minmax_v2v2_v2(fmin, fmax, e3->vert->uv);
 
                bx1 = (int)((fmin[0] - minv[0]) * invmedian);
                by1 = (int)((fmin[1] - minv[1]) * invmedian);
index fc60c5a3a0f8b4f51ef0010fe29b7c395477ba2c..efe9d1fedfe55a6ca65c5a5d5032f1472f23df16 100644 (file)
@@ -1089,7 +1089,7 @@ static void uv_map_clip_correct(Scene *scene, Object *ob, BMEditMesh *em, wmOper
 
                        BM_ITER_ELEM (l, &liter, efa, BM_LOOPS_OF_FACE) {
                                luv = CustomData_bmesh_get(&em->bm->ldata, l->head.data, CD_MLOOPUV);
-                               DO_MINMAX2(luv->uv, min, max);
+                               minmax_v2v2_v2(min, max, luv->uv);
                        }
                }