Commit: 7a0f57cd01d337f6f3a9cf02ecece635551b881d
Author: Campbell Barton
Date:   Thu Jun 4 15:28:26 2015 +1000
Branches: master
https://developer.blender.org/rB7a0f57cd01d337f6f3a9cf02ecece635551b881d

Cleanup: clarify order of precedence: &/?

===================================================================

M       source/blender/blenkernel/intern/library.c
M       source/blender/blenkernel/intern/object.c
M       source/blender/blenkernel/intern/subsurf_ccg.c
M       source/blender/blenloader/intern/versioning_250.c
M       source/blender/editors/interface/view2d.c
M       source/blender/editors/sculpt_paint/paint_cursor.c
M       source/blender/editors/sculpt_paint/paint_image_proj.c
M       source/blender/editors/sculpt_paint/sculpt.c
M       source/blender/editors/space_view3d/drawobject.c
M       source/blender/editors/transform/transform_manipulator.c
M       source/blender/modifiers/intern/MOD_explode.c
M       source/blender/modifiers/intern/MOD_screw.c
M       source/blender/render/intern/raytrace/rayobject_instance.cpp

===================================================================

diff --git a/source/blender/blenkernel/intern/library.c 
b/source/blender/blenkernel/intern/library.c
index 5546fb4..bf6f7a5 100644
--- a/source/blender/blenkernel/intern/library.c
+++ b/source/blender/blenkernel/intern/library.c
@@ -1610,7 +1610,7 @@ void rename_id(ID *id, const char *name)
 void name_uiprefix_id(char *name, const ID *id)
 {
        name[0] = id->lib ? 'L' : ' ';
-       name[1] = id->flag & LIB_FAKEUSER ? 'F' : (id->us == 0) ? '0' : ' ';
+       name[1] = (id->flag & LIB_FAKEUSER) ? 'F' : ((id->us == 0) ? '0' : ' ');
        name[2] = ' ';
 
        strcpy(name + 3, id->name + 2);
diff --git a/source/blender/blenkernel/intern/object.c 
b/source/blender/blenkernel/intern/object.c
index 939d6ee..03bf7dd 100644
--- a/source/blender/blenkernel/intern/object.c
+++ b/source/blender/blenkernel/intern/object.c
@@ -2116,7 +2116,7 @@ static void ob_parcurve(Scene *scene, Object *ob, Object 
*par, float mat[4][4])
        }
        
        /* vec: 4 items! */
-       if (where_on_path(par, ctime, vec, dir, cu->flag & CU_FOLLOW ? quat : 
NULL, &radius, NULL)) {
+       if (where_on_path(par, ctime, vec, dir, (cu->flag & CU_FOLLOW) ? quat : 
NULL, &radius, NULL)) {
 
                if (cu->flag & CU_FOLLOW) {
 #if 0
diff --git a/source/blender/blenkernel/intern/subsurf_ccg.c 
b/source/blender/blenkernel/intern/subsurf_ccg.c
index 402674c..e8bca50 100644
--- a/source/blender/blenkernel/intern/subsurf_ccg.c
+++ b/source/blender/blenkernel/intern/subsurf_ccg.c
@@ -3763,7 +3763,7 @@ struct DerivedMesh *subsurf_make_derived_from_derived(
         SubsurfFlags flags)
 {
        int useSimple = (smd->subdivType == ME_SIMPLE_SUBSURF) ? 
CCG_SIMPLE_SUBDIV : 0;
-       CCGFlags useAging = smd->flags & eSubsurfModifierFlag_DebugIncr ? 
CCG_USE_AGING : 0;
+       CCGFlags useAging = (smd->flags & eSubsurfModifierFlag_DebugIncr) ? 
CCG_USE_AGING : 0;
        int useSubsurfUv = smd->flags & eSubsurfModifierFlag_SubsurfUv;
        int drawInteriorEdges = !(smd->flags & 
eSubsurfModifierFlag_ControlEdges);
        CCGDerivedMesh *result;
diff --git a/source/blender/blenloader/intern/versioning_250.c 
b/source/blender/blenloader/intern/versioning_250.c
index 274fc47..2727f3a 100644
--- a/source/blender/blenloader/intern/versioning_250.c
+++ b/source/blender/blenloader/intern/versioning_250.c
@@ -766,7 +766,7 @@ void blo_do_versions_250(FileData *fd, Library *lib, Main 
*main)
                                        bSoundActuator *sAct = 
(bSoundActuator*) act->data;
                                        if (sAct->sound) {
                                                sound = 
blo_do_versions_newlibadr(fd, lib, sAct->sound);
-                                               sAct->flag = sound->flags & 
SOUND_FLAGS_3D ? ACT_SND_3D_SOUND : 0;
+                                               sAct->flag = (sound->flags & 
SOUND_FLAGS_3D) ? ACT_SND_3D_SOUND : 0;
                                                sAct->pitch = sound->pitch;
                                                sAct->volume = sound->volume;
                                                
sAct->sound3D.reference_distance = sound->distance;
diff --git a/source/blender/editors/interface/view2d.c 
b/source/blender/editors/interface/view2d.c
index 04ab500..4f4b5ab 100644
--- a/source/blender/editors/interface/view2d.c
+++ b/source/blender/editors/interface/view2d.c
@@ -322,7 +322,7 @@ void UI_view2d_region_reinit(View2D *v2d, short type, int 
winx, int winy)
 
                        if (do_init) {
                                float panelzoom = (style) ? style->panelzoom : 
1.0f;
-                               float scrolw = v2d->scroll & V2D_SCROLL_RIGHT ? 
V2D_SCROLL_WIDTH : 0.0f;
+                               float scrolw = (v2d->scroll & V2D_SCROLL_RIGHT) 
? V2D_SCROLL_WIDTH : 0.0f;
                                
                                v2d->tot.xmin = 0.0f;
                                v2d->tot.xmax = winx - scrolw;
diff --git a/source/blender/editors/sculpt_paint/paint_cursor.c 
b/source/blender/editors/sculpt_paint/paint_cursor.c
index 6a9d704..30ab00a 100644
--- a/source/blender/editors/sculpt_paint/paint_cursor.c
+++ b/source/blender/editors/sculpt_paint/paint_cursor.c
@@ -1026,8 +1026,8 @@ static void paint_draw_cursor(bContext *C, int x, int y, 
void *UNUSED(unused))
                /* check if brush is subtracting, use different color then */
                /* TODO: no way currently to know state of pen flip or
                 * invert key modifier without starting a stroke */
-               if ((!(ups->draw_inverted) ^
-                    !(brush->flag & BRUSH_DIR_IN)) &&
+               if (((ups->draw_inverted == 0) ^
+                    ((brush->flag & BRUSH_DIR_IN) == 0)) &&
                    ELEM(brush->sculpt_tool, SCULPT_TOOL_DRAW,
                          SCULPT_TOOL_INFLATE, SCULPT_TOOL_CLAY,
                          SCULPT_TOOL_PINCH, SCULPT_TOOL_CREASE))
diff --git a/source/blender/editors/sculpt_paint/paint_image_proj.c 
b/source/blender/editors/sculpt_paint/paint_image_proj.c
index a569bf4..58f1ebb 100644
--- a/source/blender/editors/sculpt_paint/paint_image_proj.c
+++ b/source/blender/editors/sculpt_paint/paint_image_proj.c
@@ -5085,7 +5085,7 @@ static void project_state_init(bContext *C, Object *ob, 
ProjPaintState *ps, int
                ps->blend = brush->blend;
                /* only check for inversion for the soften tool, elsewhere, a 
resident brush inversion flag can cause issues */
                if (brush->imagepaint_tool == PAINT_TOOL_SOFTEN) {
-                       ps->mode = ((ps->mode == BRUSH_STROKE_INVERT) ^ 
((brush->flag & BRUSH_DIR_IN) != 0) ?
+                       ps->mode = (((ps->mode == BRUSH_STROKE_INVERT) ^ 
((brush->flag & BRUSH_DIR_IN) != 0)) ?
                                    BRUSH_STROKE_INVERT : BRUSH_STROKE_NORMAL);
 
                        ps->blurkernel = paint_new_blur_kernel(brush, true);
diff --git a/source/blender/editors/sculpt_paint/sculpt.c 
b/source/blender/editors/sculpt_paint/sculpt.c
index 0e75a34..ede90b6 100644
--- a/source/blender/editors/sculpt_paint/sculpt.c
+++ b/source/blender/editors/sculpt_paint/sculpt.c
@@ -1200,7 +1200,7 @@ static float brush_strength(const Sculpt *sd, const 
StrokeCache *cache, const fl
        /* Primary strength input; square it to make lower values more 
sensitive */
        const float root_alpha = BKE_brush_alpha_get(scene, brush);
        float alpha        = root_alpha * root_alpha;
-       float dir          = brush->flag & BRUSH_DIR_IN ? -1 : 1;
+       float dir          = (brush->flag & BRUSH_DIR_IN) ? -1 : 1;
        float pressure     = BKE_brush_use_alpha_pressure(scene, brush) ? 
cache->pressure : 1;
        float pen_flip     = cache->pen_flip ? -1 : 1;
        float invert       = cache->invert ? -1 : 1;
diff --git a/source/blender/editors/space_view3d/drawobject.c 
b/source/blender/editors/space_view3d/drawobject.c
index c3bd657..d5a811c 100644
--- a/source/blender/editors/space_view3d/drawobject.c
+++ b/source/blender/editors/space_view3d/drawobject.c
@@ -5778,7 +5778,7 @@ static void draw_ptcache_edit(Scene *scene, View3D *v3d, 
PTCacheEdit *edit)
        PTCacheEditPoint *point;
        PTCacheEditKey *key;
        ParticleEditSettings *pset = PE_settings(scene);
-       int i, k, totpoint = edit->totpoint, timed = pset->flag & PE_FADE_TIME 
? pset->fade_frames : 0;
+       int i, k, totpoint = edit->totpoint, timed = (pset->flag & 
PE_FADE_TIME) ? pset->fade_frames : 0;
        int totkeys = 1;
        float sel_col[3];
        float nosel_col[3];
@@ -5918,7 +5918,7 @@ static void draw_ptcache_edit(Scene *scene, View3D *v3d, 
PTCacheEdit *edit)
                                                glColor3fv(nosel_col);
                                        /* has to be like this.. otherwise 
selection won't work, have try glArrayElement later..*/
                                        glBegin(GL_POINTS);
-                                       glVertex3fv(key->flag & PEK_USE_WCO ? 
key->world_co : key->co);
+                                       glVertex3fv((key->flag & PEK_USE_WCO) ? 
key->world_co : key->co);
                                        glEnd();
                                }
                        }
diff --git a/source/blender/editors/transform/transform_manipulator.c 
b/source/blender/editors/transform/transform_manipulator.c
index b55a5ff..acc6108 100644
--- a/source/blender/editors/transform/transform_manipulator.c
+++ b/source/blender/editors/transform/transform_manipulator.c
@@ -512,7 +512,7 @@ static int calc_manipulator_stats(const bContext *C)
 
                                for (k = 0, ek = point->keys; k < 
point->totkey; k++, ek++) {
                                        if (ek->flag & PEK_SELECT) {
-                                               calc_tw_center(scene, ek->flag 
& PEK_USE_WCO ? ek->world_co : ek->co);
+                                               calc_tw_center(scene, (ek->flag 
& PEK_USE_WCO) ? ek->world_co : ek->co);
                                                totsel++;
                                        }
                                }
diff --git a/source/blender/modifiers/intern/MOD_explode.c 
b/source/blender/modifiers/intern/MOD_explode.c
index fa16ebb..8260042 100644
--- a/source/blender/modifiers/intern/MOD_explode.c
+++ b/source/blender/modifiers/intern/MOD_explode.c
@@ -771,7 +771,7 @@ static DerivedMesh *cutEdges(ExplodeModifierData *emd, 
DerivedMesh *dm)
 
        for (i = 0; i < curdupface; i++) {
                mf = CDDM_get_tessface(splitdm, i);
-               test_index_face(mf, &splitdm->faceData, i, (mf->flag & 
ME_FACE_SEL ? 4 : 3));
+               test_index_face(mf, &splitdm->faceData, i, ((mf->flag & 
ME_FACE_SEL) ? 4 : 3));
        }
 
        BLI_edgehash_free(edgehash, NULL);
diff --git a/source/blender/modifiers/intern/MOD_screw.c 
b/source/blender/modifiers/intern/MOD_screw.c
index 4fa8652..ee23469 100644
--- a/source/blender/modifiers/intern/MOD_screw.c
+++ b/source/blender/modifiers/intern/MOD_screw.c
@@ -148,7 +148,7 @@ static DerivedMesh *applyModifier(ModifierData *md, Object 
*ob,
        unsigned int i, j;
        unsigned int i1, i2;
        unsigned int step_tot = use_render_params ? ltmd->render_steps : 
ltmd->steps;
-       const bool do_flip = ltmd->flag & MOD_SCREW_NORMAL_FLIP ? 1 : 0;
+       const bool do_flip = (ltmd->flag & MOD_SCREW_NORMAL_FLIP) != 0;
 
        const int quad_ord[4] = {
            do_flip ? 3 : 0,
diff --git a/source/blender/render/intern/raytrace/rayobject_instance.cpp 
b/source/blender/render/intern/raytrace/rayobject_instance.cpp
index 01e592c..d080ddc 100644
--- a/source/blender/render/intern/raytrace/rayobject_instance.cpp
+++ b/source/blender/render/intern/raytrace/rayobject_instance.cpp
@@ -197,7 +197,7 @@ static void RE_rayobject_instance_bb(RayObject *o, float 
*min, float *max)
 
        //There must be a faster way than rotating all the 8 vertexs of the BB
        for (i = 0; i < 8; i++) {
-               for (j = 0; j < 3; j++) t[j] = i & (1 << j) ? M[j] : m[j];
+               for (j = 0; j < 3; j++) t[j] = (i & (1 << j)) ? M[j] : m[j];
                mul_m4_v3(obj->target2global, t);
                DO_MINMAX(t, min, max);
        }

_______________________________________________
Bf-blender-cvs mailing list
[email protected]
http://lists.blender.org/mailman/listinfo/bf-blender-cvs

Reply via email to