Cleanup: use bool
authorCampbell Barton <ideasman42@gmail.com>
Tue, 12 Apr 2016 01:26:06 +0000 (11:26 +1000)
committerCampbell Barton <ideasman42@gmail.com>
Tue, 12 Apr 2016 01:26:06 +0000 (11:26 +1000)
source/blender/editors/curve/editcurve.c
source/blender/editors/sculpt_paint/paint_vertex.c
source/blender/imbuf/intern/jp2.c
source/blender/makesrna/intern/rna_pose.c
source/blender/makesrna/intern/rna_scene.c
source/blender/python/intern/bpy_props.c
source/blender/python/intern/bpy_rna.c
source/blender/render/intern/source/bake.c
source/blender/render/intern/source/rayshade.c

index 77fc7a832564abb0e0ddcfdc8a7949d8df9cb13a..9df611b3216d5a729d915b8be035b05629938ada 100644 (file)
@@ -1390,7 +1390,9 @@ void CURVE_OT_split(wmOperatorType *ot)
 
 /* ******************* FLAGS ********************* */
 
-static short isNurbselUV(Nurb *nu, int *u, int *v, int flag)
+static bool isNurbselUV(
+        const Nurb *nu, int flag,
+        int *r_u, int *r_v)
 {
        /* return (u != -1): 1 row in u-direction selected. U has value between 0-pntsv
         * return (v != -1): 1 column in v-direction selected. V has value between 0-pntsu
@@ -1398,7 +1400,7 @@ static short isNurbselUV(Nurb *nu, int *u, int *v, int flag)
        BPoint *bp;
        int a, b, sel;
 
-       *u = *v = -1;
+       *r_u = *r_v = -1;
 
        bp = nu->bp;
        for (b = 0; b < nu->pntsv; b++) {
@@ -1407,7 +1409,7 @@ static short isNurbselUV(Nurb *nu, int *u, int *v, int flag)
                        if (bp->f1 & flag) sel++;
                }
                if (sel == nu->pntsu) {
-                       if (*u == -1) *u = b;
+                       if (*r_u == -1) *r_u = b;
                        else return 0;
                }
                else if (sel > 1) {
@@ -1422,7 +1424,7 @@ static short isNurbselUV(Nurb *nu, int *u, int *v, int flag)
                        if (bp->f1 & flag) sel++;
                }
                if (sel == nu->pntsv) {
-                       if (*v == -1) *v = a;
+                       if (*r_v == -1) *r_v = a;
                        else return 0;
                }
                else if (sel > 1) {
@@ -1430,8 +1432,8 @@ static short isNurbselUV(Nurb *nu, int *u, int *v, int flag)
                }
        }
 
-       if (*u == -1 && *v > -1) return 1;
-       if (*v == -1 && *u > -1) return 1;
+       if (*r_u == -1 && *r_v > -1) return 1;
+       if (*r_v == -1 && *r_u > -1) return 1;
        return 0;
 }
 
@@ -1851,7 +1853,7 @@ bool ed_editnurb_extrude_flag(EditNurb *editnurb, const short flag)
                else {
                        /* which row or column is selected */
 
-                       if (isNurbselUV(nu, &u, &v, flag)) {
+                       if (isNurbselUV(nu, flag, &u, &v)) {
 
                                /* deselect all */
                                bp = nu->bp;
index bf751bfe74aaa58b3d8509622dea6b4d69f9cc76..4d3712c6e3e8e8026062746d450199fd53eeef86 100644 (file)
@@ -2979,7 +2979,7 @@ typedef struct DMGradient_userData {
        const float *sco_end;       /* [2] */
        float        sco_line_div;  /* store (1.0f / len_v2v2(sco_start, sco_end)) */
        int def_nr;
-       short is_init;
+       bool is_init;
        DMGradient_vertStore *vert_cache;
        /* only for init */
        BLI_bitmap *vert_visit;
index 570ca5ba06ec3a16118a0bbd90d18710a5d02499..390f2502ee77f38aededd038e543ef5886d623fe 100644 (file)
@@ -63,12 +63,12 @@ enum {
     DCP_CINEMA4K = 4,
 };
 
-static int check_jp2(const unsigned char *mem) /* J2K_CFMT */
+static bool check_jp2(const unsigned char *mem) /* J2K_CFMT */
 {
        return memcmp(JP2_HEAD, mem, sizeof(JP2_HEAD)) ? 0 : 1;
 }
 
-static int check_j2k(const unsigned char *mem) /* J2K_CFMT */
+static bool check_j2k(const unsigned char *mem) /* J2K_CFMT */
 {
        return memcmp(J2K_HEAD, mem, sizeof(J2K_HEAD)) ? 0 : 1;
 }
@@ -133,7 +133,7 @@ struct ImBuf *imb_jp2_decode(const unsigned char *mem, size_t size, int flags, c
        unsigned int i, i_next, w, h, planes;
        unsigned int y;
        int *r, *g, *b, *a; /* matching 'opj_image_comp.data' type */
-       int is_jp2, is_j2k;
+       bool is_jp2, is_j2k;
        
        opj_dparameters_t parameters;   /* decompression parameters */
        
index d8f1f944933d9568cfce72eec567a0279d43dde7..5bbc2b49a0d7e4fd16e2b18114ea3d16d1a857b0 100644 (file)
@@ -532,7 +532,7 @@ static bConstraint *rna_PoseChannel_constraints_new(bPoseChannel *pchan, int typ
 static void rna_PoseChannel_constraints_remove(ID *id, bPoseChannel *pchan, ReportList *reports, PointerRNA *con_ptr)
 {
        bConstraint *con = con_ptr->data;
-       const short is_ik = ELEM(con->type, CONSTRAINT_TYPE_KINEMATIC, CONSTRAINT_TYPE_SPLINEIK);
+       const bool is_ik = ELEM(con->type, CONSTRAINT_TYPE_KINEMATIC, CONSTRAINT_TYPE_SPLINEIK);
        Object *ob = (Object *)id;
 
        if (BLI_findindex(&pchan->constraints, con) == -1) {
index 94223f7f7da0f6579e5084b7052f46084edf743e..af63f6753981c8cf5bd37f9d36326aa9406ef9b4 100644 (file)
@@ -860,7 +860,7 @@ static void rna_ImageFormatSettings_file_format_set(PointerRNA *ptr, int value)
 {
        ImageFormatData *imf = (ImageFormatData *)ptr->data;
        ID *id = ptr->id.data;
-       const char is_render = (id && GS(id->name) == ID_SCE);
+       const bool is_render = (id && GS(id->name) == ID_SCE);
        /* see note below on why this is */
        const char chan_flag = BKE_imtype_valid_channels(imf->imtype, true) | (is_render ? IMA_CHAN_FLAG_BW : 0);
 
@@ -927,7 +927,7 @@ static EnumPropertyItem *rna_ImageFormatSettings_color_mode_itemf(
 {
        ImageFormatData *imf = (ImageFormatData *)ptr->data;
        ID *id = ptr->id.data;
-       const char is_render = (id && GS(id->name) == ID_SCE);
+       const bool is_render = (id && GS(id->name) == ID_SCE);
 
        /* note, we need to act differently for render
         * where 'BW' will force grayscale even if the output format writes
index ce9b3e754acbd1f9e5ba5632cca0933885a907f2..bce1d9234629af327e0041edbdd3beef6c74a106 100644 (file)
@@ -1314,7 +1314,7 @@ static int icon_id_from_name(const char *name)
        return 0;
 }
 
-static EnumPropertyItem *enum_items_from_py(PyObject *seq_fast, PyObject *def, int *defvalue, const short is_enum_flag)
+static EnumPropertyItem *enum_items_from_py(PyObject *seq_fast, PyObject *def, int *defvalue, const bool is_enum_flag)
 {
        EnumPropertyItem *items;
        PyObject *item;
index 75490a18ab0e2f8e64aec4dce59a32fe522d2368..33ac58dadb1bc087298f817344ef0608042cf5e0 100644 (file)
@@ -7363,12 +7363,8 @@ static int bpy_class_call(bContext *C, PointerRNA *ptr, FunctionRNA *func, Param
        PointerRNA funcptr;
        int err = 0, i, ret_len = 0, arg_count;
        int flag = RNA_function_flag(func);
-       const char is_staticmethod = (flag & FUNC_NO_SELF) && !(flag & FUNC_USE_SELF_TYPE);
-       const char is_classmethod = (flag & FUNC_NO_SELF) && (flag & FUNC_USE_SELF_TYPE);
-
-       /* annoying!, need to check if the screen gets set to NULL which is a
-        * hint that the file was actually re-loaded. */
-       char is_valid_wm;
+       const bool is_staticmethod = (flag & FUNC_NO_SELF) && !(flag & FUNC_USE_SELF_TYPE);
+       const bool is_classmethod = (flag & FUNC_NO_SELF) && (flag & FUNC_USE_SELF_TYPE);
 
        PropertyRNA *pret_single = NULL;
        void *retdata_single = NULL;
@@ -7395,7 +7391,9 @@ static int bpy_class_call(bContext *C, PointerRNA *ptr, FunctionRNA *func, Param
        if (C == NULL)
                C = BPy_GetContext();
 
-       is_valid_wm = (CTX_wm_manager(C) != NULL);
+       /* annoying!, need to check if the screen gets set to NULL which is a
+        * hint that the file was actually re-loaded. */
+       const bool is_valid_wm = (CTX_wm_manager(C) != NULL);
 
        bpy_context_set(C, &gilstate);
 
index b2f8c79c197abb0cf435dc94b25c42ed32f1212b..31e461b4536738016575a950f4db69c729d1d352 100644 (file)
@@ -926,7 +926,7 @@ static void *do_bake_thread(void *bs_v)
 void RE_bake_ibuf_filter(ImBuf *ibuf, char *mask, const int filter)
 {
        /* must check before filtering */
-       const short is_new_alpha = (ibuf->planes != R_IMF_PLANES_RGBA) && BKE_imbuf_alpha_test(ibuf);
+       const bool is_new_alpha = (ibuf->planes != R_IMF_PLANES_RGBA) && BKE_imbuf_alpha_test(ibuf);
 
        /* Margin */
        if (filter) {
index 0f77cd9768dafa43d94790ae454a57bfbda93302..9aac5ed1f1d5b143cab59140591bd91385c54896 100644 (file)
@@ -184,7 +184,7 @@ void freeraytree(Render *re)
 #endif
 }
 
-static int is_raytraceable_vlr(Render *re, VlakRen *vlr)
+static bool is_raytraceable_vlr(Render *re, VlakRen *vlr)
 {
        /* note: volumetric must be tracable, wire must not */
        if ((re->flag & R_BAKE_TRACE) || (vlr->flag & R_TRACEBLE) || (vlr->mat->material_type == MA_TYPE_VOLUME))
@@ -193,7 +193,7 @@ static int is_raytraceable_vlr(Render *re, VlakRen *vlr)
        return 0;
 }
 
-static int is_raytraceable(Render *re, ObjectInstanceRen *obi)
+static bool is_raytraceable(Render *re, ObjectInstanceRen *obi)
 {
        int v;
        ObjectRen *obr = obi->obr;