code cleanup: use min/max functions rather then macros.
authorCampbell Barton <ideasman42@gmail.com>
Wed, 7 Nov 2012 12:31:05 +0000 (12:31 +0000)
committerCampbell Barton <ideasman42@gmail.com>
Wed, 7 Nov 2012 12:31:05 +0000 (12:31 +0000)
20 files changed:
source/blender/blenkernel/intern/particle_system.c
source/blender/blenlib/intern/BLI_kdopbvh.c
source/blender/editors/interface/interface_layout.c
source/blender/editors/space_sequencer/sequencer_draw.c
source/blender/makesrna/intern/rna_action.c
source/blender/makesrna/intern/rna_animation.c
source/blender/makesrna/intern/rna_boid.c
source/blender/makesrna/intern/rna_curve.c
source/blender/makesrna/intern/rna_mask.c
source/blender/makesrna/intern/rna_mesh.c
source/blender/makesrna/intern/rna_modifier.c
source/blender/makesrna/intern/rna_object.c
source/blender/makesrna/intern/rna_object_force.c
source/blender/makesrna/intern/rna_particle.c
source/blender/makesrna/intern/rna_pose.c
source/blender/makesrna/intern/rna_scene.c
source/blender/makesrna/intern/rna_tracking.c
source/blender/render/intern/source/convertblender.c
source/blender/render/intern/source/pipeline.c
source/blender/render/intern/source/shadbuf.c

index 5e1e518..a780a9e 100644 (file)
@@ -467,13 +467,7 @@ static void distribute_grid(DerivedMesh *dm, ParticleSystem *psys)
        mv++;
 
        for (i=1; i<totvert; i++, mv++) {
-               min[0]=MIN2(min[0],mv->co[0]);
-               min[1]=MIN2(min[1],mv->co[1]);
-               min[2]=MIN2(min[2],mv->co[2]);
-
-               max[0]=MAX2(max[0],mv->co[0]);
-               max[1]=MAX2(max[1],mv->co[1]);
-               max[2]=MAX2(max[2],mv->co[2]);
+               minmax_v3v3_v3(min, max, mv->co);
        }
 
        sub_v3_v3v3(delta, max, min);
index 7061745..7968fca 100644 (file)
@@ -1117,7 +1117,7 @@ BVHTreeOverlap *BLI_bvhtree_overlap(BVHTree *tree1, BVHTree *tree2, unsigned int
                data[j]->overlap = (BVHTreeOverlap *)malloc(sizeof(BVHTreeOverlap) * max_ii(tree1->totleaf, tree2->totleaf));
                data[j]->tree1 = tree1;
                data[j]->tree2 = tree2;
-               data[j]->max_overlap = MAX2(tree1->totleaf, tree2->totleaf);
+               data[j]->max_overlap = max_ii(tree1->totleaf, tree2->totleaf);
                data[j]->i = 0;
                data[j]->start_axis = min_axis(tree1->start_axis, tree2->start_axis);
                data[j]->stop_axis  = min_axis(tree1->stop_axis,  tree2->stop_axis);
index 5170fc7..52d909a 100644 (file)
@@ -1780,7 +1780,7 @@ static void ui_litem_layout_row(uiLayout *litem)
 
                if (item->flag) {
                        /* fixed minimum size items */
-                       itemw = ui_item_fit(minw, fixedx, fixedw, MIN2(w, fixedw), !item->next, litem->alignment, NULL);
+                       itemw = ui_item_fit(minw, fixedx, fixedw, min_ii(w, fixedw), !item->next, litem->alignment, NULL);
                        fixedx += itemw;
                }
                else {
@@ -1959,9 +1959,9 @@ static void ui_litem_estimate_column_flow(uiLayout *litem)
                ui_item_size(item, &itemw, &itemh);
 
                y -= itemh + style->buttonspacey;
-               miny = MIN2(miny, y);
+               miny = min_ii(miny, y);
                emy -= itemh;
-               maxw = MAX2(itemw, maxw);
+               maxw = max_ii(itemw, maxw);
 
                /* decide to go to next one */
                if (col < flow->totcol - 1 && emy <= -emh) {
index 29babd3..a8103fe 100644 (file)
@@ -285,8 +285,8 @@ static void drawmeta_contents(Scene *scene, Sequence *seqm, float x1, float y1,
                drawmeta_stipple(1);
 
        for (seq = seqm->seqbase.first; seq; seq = seq->next) {
-               chan_min = MIN2(chan_min, seq->machine);
-               chan_max = MAX2(chan_max, seq->machine);
+               chan_min = min_ii(chan_min, seq->machine);
+               chan_max = max_ii(chan_max, seq->machine);
        }
 
        chan_range = (chan_max - chan_min) + 1;
index ba1aca2..1f9503f 100644 (file)
 
 #ifdef RNA_RUNTIME
 
-#include "ED_keyframing.h"
+#include "BLI_math_base.h"
+
 #include "BKE_fcurve.h"
 
+#include "ED_keyframing.h"
+
+
 static void rna_ActionGroup_channels_next(CollectionPropertyIterator *iter)
 {
        ListBaseIterator *internal = iter->internal;
@@ -191,8 +195,7 @@ static void rna_Action_active_pose_marker_index_range(PointerRNA *ptr, int *min,
        bAction *act = (bAction *)ptr->data;
 
        *min = 0;
-       *max = BLI_countlist(&act->markers) - 1;
-       *max = MAX2(0, *max);
+       *max = max_ii(0, BLI_countlist(&act->markers) - 1);
 }
 
 
index 0553f4f..350caf3 100644 (file)
@@ -54,6 +54,8 @@ EnumPropertyItem keyingset_path_grouping_items[] = {
 
 #ifdef RNA_RUNTIME
 
+#include "BLI_math_base.h"
+
 #include "BKE_animsys.h"
 #include "BKE_fcurve.h"
 #include "BKE_nla.h"
@@ -371,8 +373,7 @@ static void rna_KeyingSet_active_ksPath_index_range(PointerRNA *ptr, int *min, i
        KeyingSet *ks = (KeyingSet *)ptr->data;
 
        *min = 0;
-       *max = BLI_countlist(&ks->paths) - 1;
-       *max = MAX2(0, *max);
+       *max = max_ii(0, BLI_countlist(&ks->paths) - 1);
 }
 
 static PointerRNA rna_KeyingSet_typeinfo_get(PointerRNA *ptr)
index 8e45da5..3da718a 100644 (file)
@@ -80,6 +80,8 @@ EnumPropertyItem boidruleset_type_items[] = {
 
 #ifdef RNA_RUNTIME
 
+#include "BLI_math_base.h"
+
 #include "BKE_context.h"
 #include "BKE_depsgraph.h"
 #include "BKE_particle.h"
@@ -157,8 +159,7 @@ static void rna_BoidState_active_boid_rule_index_range(PointerRNA *ptr, int *min
 {
        BoidState *state = (BoidState *)ptr->data;
        *min = 0;
-       *max = BLI_countlist(&state->rules) - 1;
-       *max = MAX2(0, *max);
+       *max = max_ii(0, BLI_countlist(&state->rules) - 1);
 }
 
 static int rna_BoidState_active_boid_rule_index_get(PointerRNA *ptr)
@@ -224,8 +225,7 @@ static void rna_BoidSettings_active_boid_state_index_range(PointerRNA *ptr, int
 {
        BoidSettings *boids = (BoidSettings *)ptr->data;
        *min = 0;
-       *max = BLI_countlist(&boids->states) - 1;
-       *max = MAX2(0, *max);
+       *max = max_ii(0, BLI_countlist(&boids->states) - 1);
 }
 
 static int rna_BoidSettings_active_boid_state_index_get(PointerRNA *ptr)
index 35699c6..a3cff99 100644 (file)
@@ -242,16 +242,14 @@ static void rna_Curve_material_index_range(PointerRNA *ptr, int *min, int *max,
 {
        Curve *cu = (Curve *)ptr->id.data;
        *min = 0;
-       *max = cu->totcol - 1;
-       *max = MAX2(0, *max);
+       *max = max_ii(0, cu->totcol - 1);
 }
 
 static void rna_Curve_active_textbox_index_range(PointerRNA *ptr, int *min, int *max, int *softmin, int *softmax)
 {
        Curve *cu = (Curve *)ptr->id.data;
        *min = 0;
-       *max = cu->totbox - 1;
-       *max = MAX2(0, *max);
+       *max = max_ii(0, cu->totbox - 1);
 }
 
 
index 56caa25..3f23a37 100644 (file)
@@ -158,8 +158,7 @@ static void rna_Mask_layer_active_index_range(PointerRNA *ptr, int *min, int *ma
        Mask *mask = (Mask *)ptr->id.data;
 
        *min = 0;
-       *max = mask->masklay_tot - 1;
-       *max = MAX2(0, *max);
+       *max = max_ii(0, mask->masklay_tot - 1);
 
        *softmin = *min;
        *softmax = *max;
index f0b8433..170e614 100644 (file)
@@ -1010,8 +1010,7 @@ static void rna_MeshPoly_material_index_range(PointerRNA *ptr, int *min, int *ma
 {
        Mesh *me = rna_mesh(ptr);
        *min = 0;
-       *max = me->totcol - 1;
-       *max = MAX2(0, *max);
+       *max = max_ii(0, me->totcol - 1);
 }
 
 static int rna_MeshVertex_index_get(PointerRNA *ptr)
index 8eca716..d421eb6 100644 (file)
@@ -481,8 +481,7 @@ static void rna_MultiresModifier_level_range(PointerRNA *ptr, int *min, int *max
        MultiresModifierData *mmd = (MultiresModifierData *)ptr->data;
 
        *min = 0;
-       *max = mmd->totlvl; /* intentionally _not_ -1 */
-       *max = MAX2(0, *max);
+       *max = max_ii(0, mmd->totlvl);  /* intentionally _not_ -1 */
 }
 
 static int rna_MultiresModifier_external_get(PointerRNA *ptr)
index 10f361e..82f27b9 100644 (file)
@@ -547,8 +547,7 @@ static void rna_Object_active_vertex_group_index_range(PointerRNA *ptr, int *min
        Object *ob = (Object *)ptr->id.data;
 
        *min = 0;
-       *max = BLI_countlist(&ob->defbase) - 1;
-       *max = MAX2(0, *max);
+       *max = max_ii(0, BLI_countlist(&ob->defbase) - 1);
 }
 
 void rna_object_vgroup_name_index_get(PointerRNA *ptr, char *value, int index)
@@ -658,7 +657,7 @@ static void rna_Object_active_material_index_range(PointerRNA *ptr, int *min, in
 {
        Object *ob = (Object *)ptr->id.data;
        *min = 0;
-       *max = MAX2(ob->totcol - 1, 0);
+       *max = max_ii(ob->totcol - 1, 0);
 }
 
 /* returns active base material */
@@ -684,8 +683,7 @@ static void rna_Object_active_particle_system_index_range(PointerRNA *ptr, int *
 {
        Object *ob = (Object *)ptr->id.data;
        *min = 0;
-       *max = BLI_countlist(&ob->particlesystem) - 1;
-       *max = MAX2(0, *max);
+       *max = max_ii(0, BLI_countlist(&ob->particlesystem) - 1);
 }
 
 static int rna_Object_active_particle_system_index_get(PointerRNA *ptr)
index f47b864..6d7187d 100644 (file)
@@ -51,6 +51,9 @@ static EnumPropertyItem effector_shape_items[] = {
 
 #ifdef RNA_RUNTIME
 
+#include "BLI_math_base.h"
+
+
 /* type specific return values only used from functions */
 static EnumPropertyItem curve_shape_items[] = {
        {PFIELD_SHAPE_POINT, "POINT", 0, "Point", ""},
@@ -239,8 +242,7 @@ static void rna_Cache_active_point_cache_index_range(PointerRNA *ptr, int *min,
 
        for (pid = pidlist.first; pid; pid = pid->next) {
                if (pid->cache == cache) {
-                       *max = BLI_countlist(pid->ptcaches) - 1;
-                       *max = MAX2(0, *max);
+                       *max = max_ii(0, BLI_countlist(pid->ptcaches) - 1);
                        break;
                }
        }
index 9ba2a06..6825d3d 100644 (file)
@@ -530,8 +530,7 @@ static void rna_ParticleSystem_active_particle_target_index_range(PointerRNA *pt
 {
        ParticleSystem *psys = (ParticleSystem *)ptr->data;
        *min = 0;
-       *max = BLI_countlist(&psys->targets) - 1;
-       *max = MAX2(0, *max);
+       *max = max_ii(0, BLI_countlist(&psys->targets) - 1);
 }
 
 static int rna_ParticleSystem_active_particle_target_index_get(PointerRNA *ptr)
@@ -670,8 +669,7 @@ static void rna_ParticleDupliWeight_active_index_range(PointerRNA *ptr, int *min
 {
        ParticleSettings *part = (ParticleSettings *)ptr->id.data;
        *min = 0;
-       *max = BLI_countlist(&part->dupliweights) - 1;
-       *max = MAX2(0, *max);
+       *max = max_ii(0, BLI_countlist(&part->dupliweights) - 1);
 }
 
 static int rna_ParticleDupliWeight_active_index_get(PointerRNA *ptr)
index c295373..412aad2 100644 (file)
@@ -381,13 +381,7 @@ static void rna_PoseChannel_bone_group_index_range(PointerRNA *ptr, int *min, in
        bPose *pose = (ob) ? ob->pose : NULL;
        
        *min = 0;
-       
-       if (pose) {
-               *max = BLI_countlist(&pose->agroups) - 1;
-               *max = MAX2(0, *max);
-       }
-       else
-               *max = 0;
+       *max = pose ? max_ii(0, BLI_countlist(&pose->agroups) - 1) : 0;
 }
 
 static PointerRNA rna_Pose_active_bone_group_get(PointerRNA *ptr)
@@ -419,8 +413,7 @@ static void rna_Pose_active_bone_group_index_range(PointerRNA *ptr, int *min, in
        bPose *pose = (bPose *)ptr->data;
 
        *min = 0;
-       *max = BLI_countlist(&pose->agroups) - 1;
-       *max = MAX2(0, *max);
+       *max = max_ii(0, BLI_countlist(&pose->agroups) - 1);
 }
 
 #if 0
index 69bb1cc..b102d46 100644 (file)
@@ -1002,8 +1002,7 @@ static void rna_RenderSettings_active_layer_index_range(PointerRNA *ptr, int *mi
        RenderData *rd = (RenderData *)ptr->data;
 
        *min = 0;
-       *max = BLI_countlist(&rd->layers) - 1;
-       *max = MAX2(0, *max);
+       *max = max_ii(0, BLI_countlist(&rd->layers) - 1);
 }
 
 static PointerRNA rna_RenderSettings_active_layer_get(PointerRNA *ptr)
index 51b50f7..074a8ed 100644 (file)
@@ -120,8 +120,7 @@ static void rna_tracking_active_object_index_range(PointerRNA *ptr, int *min, in
        MovieClip *clip = (MovieClip *)ptr->id.data;
 
        *min = 0;
-       *max = clip->tracking.tot_object - 1;
-       *max = MAX2(0, *max);
+       *max = max_ii(0, clip->tracking.tot_object - 1);
 }
 
 static PointerRNA rna_tracking_active_track_get(PointerRNA *ptr)
@@ -263,8 +262,7 @@ static void rna_tracking_stabTracks_active_index_range(PointerRNA *ptr, int *min
        MovieClip *clip = (MovieClip *)ptr->id.data;
 
        *min = 0;
-       *max = clip->tracking.stabilization.tot_track - 1;
-       *max = MAX2(0, *max);
+       *max = max_ii(0, clip->tracking.stabilization.tot_track - 1);
 }
 
 static void rna_tracking_flushUpdate(Main *UNUSED(bmain), Scene *scene, PointerRNA *ptr)
index e305f43..eb0a13b 100644 (file)
@@ -4296,7 +4296,7 @@ static void finalize_render_object(Render *re, ObjectRen *obr, int timeoffset)
                                
                                /* compute average bounding box of strandpoint itself (width) */
                                if (obr->strandbuf->flag & R_STRAND_B_UNITS)
-                                       obr->strandbuf->maxwidth= MAX2(obr->strandbuf->ma->strand_sta, obr->strandbuf->ma->strand_end);
+                                       obr->strandbuf->maxwidth = max_ff(obr->strandbuf->ma->strand_sta, obr->strandbuf->ma->strand_end);
                                else
                                        obr->strandbuf->maxwidth= 0.0f;
                                
index 7ef3aec..d305ed0 100644 (file)
@@ -2507,8 +2507,8 @@ int RE_WriteEnvmapResult(struct ReportList *reports, Scene *scene, EnvMap *env,
 
        if (env->type == ENV_CUBE) {
                for (i = 0; i < 12; i += 2) {
-                       maxX = MAX2(maxX, layout[i] + 1);
-                       maxY = MAX2(maxY, layout[i + 1] + 1);
+                       maxX = max_ii(maxX, (int)layout[i] + 1);
+                       maxY = max_ii(maxY, (int)layout[i + 1] + 1);
                }
 
                ibuf = IMB_allocImBuf(maxX * dx, maxY * dx, 24, IB_rectfloat);
index c37f48b..a7f6b40 100644 (file)
@@ -637,7 +637,7 @@ static void shadowbuf_autoclip(Render *re, LampRen *lar)
 
        maxtotvert= 0;
        for (obr=re->objecttable.first; obr; obr=obr->next)
-               maxtotvert= MAX2(obr->totvert, maxtotvert);
+               maxtotvert = max_ii(obr->totvert, maxtotvert);
 
        clipflag= MEM_callocN(sizeof(char)*maxtotvert, "autoclipflag");