Cleanup: remove redundant parentheses
This commit is contained in:
parent
09b89da349
commit
bf5b1fa726
|
@ -49,22 +49,22 @@ struct bGPDlayer_Mask;
|
|||
struct bGPDstroke;
|
||||
struct bGPdata;
|
||||
|
||||
#define GPENCIL_SIMPLIFY(scene) ((scene->r.simplify_gpencil & SIMPLIFY_GPENCIL_ENABLE))
|
||||
#define GPENCIL_SIMPLIFY(scene) (scene->r.simplify_gpencil & SIMPLIFY_GPENCIL_ENABLE)
|
||||
#define GPENCIL_SIMPLIFY_ONPLAY(playing) \
|
||||
(((playing == true) && (scene->r.simplify_gpencil & SIMPLIFY_GPENCIL_ON_PLAY)) || \
|
||||
((scene->r.simplify_gpencil & SIMPLIFY_GPENCIL_ON_PLAY) == 0))
|
||||
#define GPENCIL_SIMPLIFY_FILL(scene, playing) \
|
||||
((GPENCIL_SIMPLIFY_ONPLAY(playing) && (GPENCIL_SIMPLIFY(scene)) && \
|
||||
((GPENCIL_SIMPLIFY_ONPLAY(playing) && GPENCIL_SIMPLIFY(scene) && \
|
||||
(scene->r.simplify_gpencil & SIMPLIFY_GPENCIL_FILL)))
|
||||
#define GPENCIL_SIMPLIFY_MODIF(scene) \
|
||||
((GPENCIL_SIMPLIFY(scene) && (scene->r.simplify_gpencil & SIMPLIFY_GPENCIL_MODIFIER)))
|
||||
#define GPENCIL_SIMPLIFY_FX(scene, playing) \
|
||||
((GPENCIL_SIMPLIFY_ONPLAY(playing) && (GPENCIL_SIMPLIFY(scene)) && \
|
||||
((GPENCIL_SIMPLIFY_ONPLAY(playing) && GPENCIL_SIMPLIFY(scene) && \
|
||||
(scene->r.simplify_gpencil & SIMPLIFY_GPENCIL_FX)))
|
||||
#define GPENCIL_SIMPLIFY_TINT(scene) \
|
||||
((GPENCIL_SIMPLIFY(scene)) && (scene->r.simplify_gpencil & SIMPLIFY_GPENCIL_TINT))
|
||||
(GPENCIL_SIMPLIFY(scene) && (scene->r.simplify_gpencil & SIMPLIFY_GPENCIL_TINT))
|
||||
#define GPENCIL_SIMPLIFY_AA(scene) \
|
||||
((GPENCIL_SIMPLIFY(scene)) && (scene->r.simplify_gpencil & SIMPLIFY_GPENCIL_AA))
|
||||
(GPENCIL_SIMPLIFY(scene) && (scene->r.simplify_gpencil & SIMPLIFY_GPENCIL_AA))
|
||||
|
||||
/* Vertex Color macros. */
|
||||
#define GPENCIL_USE_VERTEX_COLOR(toolsettings) \
|
||||
|
|
|
@ -688,7 +688,7 @@ static float fcm_cycles_time(
|
|||
ofs = lastkey[0];
|
||||
}
|
||||
}
|
||||
if ((ELEM(0, side, mode))) {
|
||||
if (ELEM(0, side, mode)) {
|
||||
return evaltime;
|
||||
}
|
||||
|
||||
|
|
|
@ -2730,7 +2730,7 @@ void BKE_gpencil_visible_stroke_advanced_iter(ViewLayer *view_layer,
|
|||
int cfra)
|
||||
{
|
||||
bGPdata *gpd = (bGPdata *)ob->data;
|
||||
const bool is_multiedit = ((GPENCIL_MULTIEDIT_SESSIONS_ON(gpd)) && (!GPENCIL_PLAY_ON(gpd)));
|
||||
const bool is_multiedit = (GPENCIL_MULTIEDIT_SESSIONS_ON(gpd) && (!GPENCIL_PLAY_ON(gpd)));
|
||||
const bool is_onion = do_onion && ((gpd->flag & GP_DATA_STROKE_WEIGHTMODE) == 0);
|
||||
const bool is_drawing = (gpd->runtime.sbuffer_used > 0);
|
||||
|
||||
|
|
|
@ -273,7 +273,7 @@ int BKE_gpencil_time_modifier_cfra(Depsgraph *depsgraph,
|
|||
if (GPENCIL_MODIFIER_ACTIVE(md, is_render)) {
|
||||
const GpencilModifierTypeInfo *mti = BKE_gpencil_modifier_get_info(md->type);
|
||||
|
||||
if ((GPENCIL_MODIFIER_EDIT(md, is_edit)) && (!is_render)) {
|
||||
if (GPENCIL_MODIFIER_EDIT(md, is_edit) && (!is_render)) {
|
||||
continue;
|
||||
}
|
||||
|
||||
|
@ -838,7 +838,7 @@ void BKE_gpencil_modifiers_calc(Depsgraph *depsgraph, Scene *scene, Object *ob)
|
|||
if (GPENCIL_MODIFIER_ACTIVE(md, is_render)) {
|
||||
const GpencilModifierTypeInfo *mti = BKE_gpencil_modifier_get_info(md->type);
|
||||
|
||||
if ((GPENCIL_MODIFIER_EDIT(md, is_edit)) && (!is_render)) {
|
||||
if (GPENCIL_MODIFIER_EDIT(md, is_edit) && (!is_render)) {
|
||||
continue;
|
||||
}
|
||||
|
||||
|
|
|
@ -937,7 +937,7 @@ void BKE_maskrasterize_handle_init(MaskRasterHandle *mr_handle,
|
|||
ListBase isect_remedgebase = {NULL, NULL};
|
||||
|
||||
/* now we have all the splines */
|
||||
face_coords = MEM_mallocN((sizeof(float[3])) * sf_vert_tot, "maskrast_face_coords");
|
||||
face_coords = MEM_mallocN(sizeof(float[3]) * sf_vert_tot, "maskrast_face_coords");
|
||||
|
||||
/* init bounds */
|
||||
BLI_rctf_init_minmax(&bounds);
|
||||
|
|
|
@ -312,7 +312,7 @@ static void mesh_blend_read_data(BlendDataReader *reader, ID *id)
|
|||
mesh->totselect = 0;
|
||||
}
|
||||
|
||||
if ((BLO_read_requires_endian_switch(reader)) && mesh->tface) {
|
||||
if (BLO_read_requires_endian_switch(reader) && mesh->tface) {
|
||||
TFace *tf = mesh->tface;
|
||||
for (int i = 0; i < mesh->totface; i++, tf++) {
|
||||
BLI_endian_switch_uint32_array(tf->col, 4);
|
||||
|
|
|
@ -506,7 +506,7 @@ void ntreeBlendWrite(BlendWriter *writer, bNodeTree *ntree)
|
|||
|
||||
if (node->storage) {
|
||||
/* could be handlerized at some point, now only 1 exception still */
|
||||
if ((ELEM(ntree->type, NTREE_SHADER, NTREE_GEOMETRY)) &&
|
||||
if (ELEM(ntree->type, NTREE_SHADER, NTREE_GEOMETRY) &&
|
||||
ELEM(node->type, SH_NODE_CURVE_VEC, SH_NODE_CURVE_RGB)) {
|
||||
BKE_curvemapping_blend_write(writer, (const CurveMapping *)node->storage);
|
||||
}
|
||||
|
@ -3143,7 +3143,7 @@ static void free_localized_node_groups(bNodeTree *ntree)
|
|||
}
|
||||
|
||||
LISTBASE_FOREACH (bNode *, node, &ntree->nodes) {
|
||||
if ((ELEM(node->type, NODE_GROUP, NODE_CUSTOM_GROUP)) && node->id) {
|
||||
if (ELEM(node->type, NODE_GROUP, NODE_CUSTOM_GROUP) && node->id) {
|
||||
bNodeTree *ngroup = (bNodeTree *)node->id;
|
||||
ntreeFreeTree(ngroup);
|
||||
MEM_freeN(ngroup);
|
||||
|
@ -3335,7 +3335,7 @@ bNodeTree *ntreeLocalize(bNodeTree *ntree)
|
|||
ltree->id.tag |= LIB_TAG_LOCALIZED;
|
||||
|
||||
LISTBASE_FOREACH (bNode *, node, <ree->nodes) {
|
||||
if ((ELEM(node->type, NODE_GROUP, NODE_CUSTOM_GROUP)) && node->id) {
|
||||
if (ELEM(node->type, NODE_GROUP, NODE_CUSTOM_GROUP) && node->id) {
|
||||
node->id = (ID *)ntreeLocalize((bNodeTree *)node->id);
|
||||
}
|
||||
}
|
||||
|
|
|
@ -184,7 +184,7 @@ size_t BLI_system_memory_max_in_megabytes(void)
|
|||
/* Maximum addressable bytes on this platform.
|
||||
*
|
||||
* NOTE: Due to the shift arithmetic this is a half of the memory. */
|
||||
const size_t limit_bytes_half = (((size_t)1) << ((sizeof(size_t[8])) - 1));
|
||||
const size_t limit_bytes_half = (((size_t)1) << (sizeof(size_t[8]) - 1));
|
||||
/* Convert it to megabytes and return. */
|
||||
return (limit_bytes_half >> 20) * 2;
|
||||
}
|
||||
|
|
|
@ -800,7 +800,7 @@ short BM_edge_flag_to_mflag(BMEdge *e)
|
|||
((hflag & BM_ELEM_DRAW) ? ME_EDGEDRAW : 0) |
|
||||
((hflag & BM_ELEM_SMOOTH) == 0 ? ME_SHARP : 0) |
|
||||
((hflag & BM_ELEM_HIDDEN) ? ME_HIDE : 0) |
|
||||
((BM_edge_is_wire(e)) ? ME_LOOSEEDGE : 0) | /* not typical */
|
||||
(BM_edge_is_wire(e) ? ME_LOOSEEDGE : 0) | /* not typical */
|
||||
ME_EDGERENDER);
|
||||
}
|
||||
char BM_face_flag_to_mflag(BMFace *f)
|
||||
|
|
|
@ -337,7 +337,7 @@ static bool bm_face_split_edgenet_find_loop_walk(BMVert *v_init,
|
|||
/* in rare cases there may be edges with a single connecting vertex */
|
||||
if (e_next != e_first) {
|
||||
do {
|
||||
if ((BM_ELEM_API_FLAG_TEST(e_next, EDGE_NET)) &&
|
||||
if (BM_ELEM_API_FLAG_TEST(e_next, EDGE_NET) &&
|
||||
(bm_edge_flagged_radial_count(e_next) < 2)) {
|
||||
BMVert *v_next;
|
||||
|
||||
|
|
|
@ -842,7 +842,7 @@ static void *bmw_IslandManifoldWalker_step(BMWalker *walker)
|
|||
/* utility function to see if an edge is a part of an ngon boundary */
|
||||
static bool bm_edge_is_single(BMEdge *e)
|
||||
{
|
||||
return ((BM_edge_is_boundary(e)) && (e->l->f->len > 4) &&
|
||||
return (BM_edge_is_boundary(e) && (e->l->f->len > 4) &&
|
||||
(BM_edge_is_boundary(e->l->next->e) || BM_edge_is_boundary(e->l->prev->e)));
|
||||
}
|
||||
|
||||
|
|
|
@ -51,10 +51,10 @@ static int bm_edge_length_cmp(const void *a_, const void *b_)
|
|||
const BMEdge *e_a = *(const void **)a_;
|
||||
const BMEdge *e_b = *(const void **)b_;
|
||||
|
||||
int e_a_concave = ((BM_elem_flag_test(e_a->v1, BM_ELEM_TAG)) &&
|
||||
(BM_elem_flag_test(e_a->v2, BM_ELEM_TAG)));
|
||||
int e_b_concave = ((BM_elem_flag_test(e_b->v1, BM_ELEM_TAG)) &&
|
||||
(BM_elem_flag_test(e_b->v2, BM_ELEM_TAG)));
|
||||
int e_a_concave = (BM_elem_flag_test(e_a->v1, BM_ELEM_TAG) &&
|
||||
BM_elem_flag_test(e_a->v2, BM_ELEM_TAG));
|
||||
int e_b_concave = (BM_elem_flag_test(e_b->v1, BM_ELEM_TAG) &&
|
||||
BM_elem_flag_test(e_b->v2, BM_ELEM_TAG));
|
||||
|
||||
/* merge edges between concave edges last since these
|
||||
* are most likely to remain and be the main dividers */
|
||||
|
|
|
@ -93,7 +93,7 @@ static BMEdge *edge_next(BMesh *bm, BMEdge *e)
|
|||
|
||||
for (i = 0; i < 2; i++) {
|
||||
BM_ITER_ELEM (e2, &iter, i ? e->v2 : e->v1, BM_EDGES_OF_VERT) {
|
||||
if ((BMO_edge_flag_test(bm, e2, EDGE_MARK)) &&
|
||||
if (BMO_edge_flag_test(bm, e2, EDGE_MARK) &&
|
||||
(BMO_edge_flag_test(bm, e2, EDGE_VIS) == false) && (e2 != e)) {
|
||||
return e2;
|
||||
}
|
||||
|
|
|
@ -645,20 +645,20 @@ void bmo_grid_fill_exec(BMesh *bm, BMOperator *op)
|
|||
bm_edgeloop_flag_set(estore_a, BM_ELEM_HIDDEN, true);
|
||||
bm_edgeloop_flag_set(estore_b, BM_ELEM_HIDDEN, true);
|
||||
|
||||
if ((BM_mesh_edgeloops_find_path(
|
||||
bm, &eloops_rail, bm_edge_test_rail_cb, bm, v_a_first, v_b_first)) &&
|
||||
(BM_mesh_edgeloops_find_path(
|
||||
bm, &eloops_rail, bm_edge_test_rail_cb, bm, v_a_last, v_b_last))) {
|
||||
if (BM_mesh_edgeloops_find_path(
|
||||
bm, &eloops_rail, bm_edge_test_rail_cb, bm, v_a_first, v_b_first) &&
|
||||
BM_mesh_edgeloops_find_path(
|
||||
bm, &eloops_rail, bm_edge_test_rail_cb, bm, v_a_last, v_b_last)) {
|
||||
estore_rail_a = eloops_rail.first;
|
||||
estore_rail_b = eloops_rail.last;
|
||||
}
|
||||
else {
|
||||
BM_mesh_edgeloops_free(&eloops_rail);
|
||||
|
||||
if ((BM_mesh_edgeloops_find_path(
|
||||
bm, &eloops_rail, bm_edge_test_rail_cb, bm, v_a_first, v_b_last)) &&
|
||||
(BM_mesh_edgeloops_find_path(
|
||||
bm, &eloops_rail, bm_edge_test_rail_cb, bm, v_a_last, v_b_first))) {
|
||||
if (BM_mesh_edgeloops_find_path(
|
||||
bm, &eloops_rail, bm_edge_test_rail_cb, bm, v_a_first, v_b_last) &&
|
||||
BM_mesh_edgeloops_find_path(
|
||||
bm, &eloops_rail, bm_edge_test_rail_cb, bm, v_a_last, v_b_first)) {
|
||||
estore_rail_a = eloops_rail.first;
|
||||
estore_rail_b = eloops_rail.last;
|
||||
BM_edgeloop_flip(bm, estore_b);
|
||||
|
|
|
@ -860,12 +860,12 @@ static bool bm_edgering_pair_order_is_flipped(BMesh *UNUSED(bm),
|
|||
/* step around any fan-faces on both sides */
|
||||
do {
|
||||
v_iter_a_step = v_iter_a_step->next;
|
||||
} while (v_iter_a_step && ((BM_edge_exists(v_iter_a_step->data, v_iter_b_first->data)) ||
|
||||
(BM_edge_exists(v_iter_a_step->data, v_iter_b_first->next->data))));
|
||||
} while (v_iter_a_step && (BM_edge_exists(v_iter_a_step->data, v_iter_b_first->data) ||
|
||||
BM_edge_exists(v_iter_a_step->data, v_iter_b_first->next->data)));
|
||||
do {
|
||||
v_iter_b_step = v_iter_b_step->next;
|
||||
} while (v_iter_b_step && ((BM_edge_exists(v_iter_b_step->data, v_iter_a_first->data)) ||
|
||||
(BM_edge_exists(v_iter_b_step->data, v_iter_a_first->next->data))));
|
||||
} while (v_iter_b_step && (BM_edge_exists(v_iter_b_step->data, v_iter_a_first->data) ||
|
||||
BM_edge_exists(v_iter_b_step->data, v_iter_a_first->next->data)));
|
||||
|
||||
v_iter_a_step = v_iter_a_step ? v_iter_a_step->prev : lb_a->last;
|
||||
v_iter_b_step = v_iter_b_step ? v_iter_b_step->prev : lb_b->last;
|
||||
|
|
|
@ -91,7 +91,7 @@ typedef struct BoundSphere {
|
|||
typedef char DRWViewportEmptyList;
|
||||
|
||||
#define DRW_VIEWPORT_LIST_SIZE(list) \
|
||||
(sizeof(list) == sizeof(DRWViewportEmptyList) ? 0 : ((sizeof(list)) / sizeof(void *)))
|
||||
(sizeof(list) == sizeof(DRWViewportEmptyList) ? 0 : (sizeof(list) / sizeof(void *)))
|
||||
|
||||
/* Unused members must be either pass list or 'char *' when not used. */
|
||||
#define DRW_VIEWPORT_DATA_SIZE(ty) \
|
||||
|
|
|
@ -278,7 +278,7 @@ static void gpencil_stroke_pair_table(bContext *C,
|
|||
tGPDinterpolate_layer *tgpil)
|
||||
{
|
||||
bGPdata *gpd = tgpi->gpd;
|
||||
const bool only_selected = ((GPENCIL_EDIT_MODE(gpd)) &&
|
||||
const bool only_selected = (GPENCIL_EDIT_MODE(gpd) &&
|
||||
((tgpi->flag & GP_TOOLFLAG_INTERPOLATE_ONLY_SELECTED) != 0));
|
||||
const bool is_multiedit = (bool)GPENCIL_MULTIEDIT_SESSIONS_ON(gpd);
|
||||
|
||||
|
@ -291,8 +291,7 @@ static void gpencil_stroke_pair_table(bContext *C,
|
|||
LISTBASE_FOREACH (bGPDstroke *, gps_from, &tgpil->prevFrame->strokes) {
|
||||
bGPDstroke *gps_to = NULL;
|
||||
/* only selected */
|
||||
if ((GPENCIL_EDIT_MODE(gpd)) && (only_selected) &&
|
||||
((gps_from->flag & GP_STROKE_SELECT) == 0)) {
|
||||
if (GPENCIL_EDIT_MODE(gpd) && (only_selected) && ((gps_from->flag & GP_STROKE_SELECT) == 0)) {
|
||||
continue;
|
||||
}
|
||||
/* skip strokes that are invalid for current view */
|
||||
|
@ -712,7 +711,7 @@ static bool gpencil_interpolate_set_init_values(bContext *C, wmOperator *op, tGP
|
|||
tgpi->flag, (RNA_enum_get(op->ptr, "layers") == 1), GP_TOOLFLAG_INTERPOLATE_ALL_LAYERS);
|
||||
SET_FLAG_FROM_TEST(
|
||||
tgpi->flag,
|
||||
((GPENCIL_EDIT_MODE(tgpi->gpd)) && (RNA_boolean_get(op->ptr, "interpolate_selected_only"))),
|
||||
(GPENCIL_EDIT_MODE(tgpi->gpd) && (RNA_boolean_get(op->ptr, "interpolate_selected_only"))),
|
||||
GP_TOOLFLAG_INTERPOLATE_ONLY_SELECTED);
|
||||
|
||||
tgpi->flipmode = RNA_enum_get(op->ptr, "flip");
|
||||
|
@ -1249,7 +1248,7 @@ static int gpencil_interpolate_seq_exec(bContext *C, wmOperator *op)
|
|||
const int step = RNA_int_get(op->ptr, "step");
|
||||
const bool is_multiedit = (bool)GPENCIL_MULTIEDIT_SESSIONS_ON(gpd);
|
||||
const bool all_layers = (bool)(RNA_enum_get(op->ptr, "layers") == 1);
|
||||
const bool only_selected = ((GPENCIL_EDIT_MODE(gpd)) &&
|
||||
const bool only_selected = (GPENCIL_EDIT_MODE(gpd) &&
|
||||
(RNA_boolean_get(op->ptr, "interpolate_selected_only") != 0));
|
||||
|
||||
eGP_InterpolateFlipMode flipmode = RNA_enum_get(op->ptr, "flip");
|
||||
|
@ -1309,7 +1308,7 @@ static int gpencil_interpolate_seq_exec(bContext *C, wmOperator *op)
|
|||
LISTBASE_FOREACH (bGPDstroke *, gps_from, &prevFrame->strokes) {
|
||||
bGPDstroke *gps_to = NULL;
|
||||
/* Only selected. */
|
||||
if ((GPENCIL_EDIT_MODE(gpd)) && (only_selected) &&
|
||||
if (GPENCIL_EDIT_MODE(gpd) && (only_selected) &&
|
||||
((gps_from->flag & GP_STROKE_SELECT) == 0)) {
|
||||
continue;
|
||||
}
|
||||
|
|
|
@ -1352,7 +1352,7 @@ static void gpencil_sculpt_brush_init_stroke(bContext *C, tGP_BrushEditData *gso
|
|||
* - This is useful when animating as it saves that "uh-oh" moment when you realize you've
|
||||
* spent too much time editing the wrong frame.
|
||||
*/
|
||||
if ((IS_AUTOKEY_ON(scene)) && (gpf->framenum != cfra)) {
|
||||
if (IS_AUTOKEY_ON(scene) && (gpf->framenum != cfra)) {
|
||||
BKE_gpencil_frame_addcopy(gpl, cfra);
|
||||
/* Need tag to recalculate evaluated data to avoid crashes. */
|
||||
DEG_id_tag_update(&gso->gpd->id, ID_RECALC_GEOMETRY | ID_RECALC_COPY_ON_WRITE);
|
||||
|
@ -1377,7 +1377,7 @@ static float gpencil_sculpt_rotation_eval_get(tGP_BrushEditData *gso,
|
|||
int idx_eval)
|
||||
{
|
||||
/* If multiframe or no modifiers, return 0. */
|
||||
if ((GPENCIL_MULTIEDIT_SESSIONS_ON(gso->gpd)) || (!gso->is_transformed)) {
|
||||
if (GPENCIL_MULTIEDIT_SESSIONS_ON(gso->gpd) || (!gso->is_transformed)) {
|
||||
return 0.0f;
|
||||
}
|
||||
|
||||
|
@ -1513,8 +1513,7 @@ static bool gpencil_sculpt_brush_do_stroke(tGP_BrushEditData *gso,
|
|||
}
|
||||
pt_active = (pt->runtime.pt_orig) ? pt->runtime.pt_orig : pt;
|
||||
/* If masked and the point is not selected, skip it. */
|
||||
if ((GPENCIL_ANY_SCULPT_MASK(gso->mask)) &&
|
||||
((pt_active->flag & GP_SPOINT_SELECT) == 0)) {
|
||||
if (GPENCIL_ANY_SCULPT_MASK(gso->mask) && ((pt_active->flag & GP_SPOINT_SELECT) == 0)) {
|
||||
continue;
|
||||
}
|
||||
index = (pt->runtime.pt_orig) ? pt->runtime.idx_orig : i;
|
||||
|
|
|
@ -919,7 +919,7 @@ static bool gpencil_vertexpaint_select_stroke(tGP_BrushVertexpaintData *gso,
|
|||
pt_active = pt->runtime.pt_orig;
|
||||
if (pt_active != NULL) {
|
||||
/* If masked and the point is not selected, skip it. */
|
||||
if ((GPENCIL_ANY_VERTEX_MASK(gso->mask)) &&
|
||||
if (GPENCIL_ANY_VERTEX_MASK(gso->mask) &&
|
||||
((pt_active->flag & GP_SPOINT_SELECT) == 0)) {
|
||||
continue;
|
||||
}
|
||||
|
|
|
@ -2453,7 +2453,7 @@ bool ui_but_is_rna_valid(uiBut *but)
|
|||
*/
|
||||
bool ui_but_supports_cycling(const uiBut *but)
|
||||
{
|
||||
return ((ELEM(but->type, UI_BTYPE_ROW, UI_BTYPE_NUM, UI_BTYPE_NUM_SLIDER, UI_BTYPE_LISTBOX)) ||
|
||||
return (ELEM(but->type, UI_BTYPE_ROW, UI_BTYPE_NUM, UI_BTYPE_NUM_SLIDER, UI_BTYPE_LISTBOX) ||
|
||||
(but->type == UI_BTYPE_MENU && ui_but_menu_step_poll(but)) ||
|
||||
(but->type == UI_BTYPE_COLOR && ((uiButColor *)but)->is_pallete_color) ||
|
||||
(but->menu_step_func != NULL));
|
||||
|
|
|
@ -688,13 +688,13 @@ static void ui_searchbox_region_draw_cb(const bContext *C, ARegion *region)
|
|||
if (data->items.more) {
|
||||
ui_searchbox_butrect(&rect, data, data->items.maxitem - 1);
|
||||
GPU_blend(GPU_BLEND_ALPHA);
|
||||
UI_icon_draw((BLI_rcti_size_x(&rect)) / 2, rect.ymin - 9, ICON_TRIA_DOWN);
|
||||
UI_icon_draw(BLI_rcti_size_x(&rect) / 2, rect.ymin - 9, ICON_TRIA_DOWN);
|
||||
GPU_blend(GPU_BLEND_NONE);
|
||||
}
|
||||
if (data->items.offset) {
|
||||
ui_searchbox_butrect(&rect, data, 0);
|
||||
GPU_blend(GPU_BLEND_ALPHA);
|
||||
UI_icon_draw((BLI_rcti_size_x(&rect)) / 2, rect.ymax - 7, ICON_TRIA_UP);
|
||||
UI_icon_draw(BLI_rcti_size_x(&rect) / 2, rect.ymax - 7, ICON_TRIA_UP);
|
||||
GPU_blend(GPU_BLEND_NONE);
|
||||
}
|
||||
}
|
||||
|
@ -990,13 +990,13 @@ static void ui_searchbox_region_draw_cb__operator(const bContext *UNUSED(C), ARe
|
|||
if (data->items.more) {
|
||||
ui_searchbox_butrect(&rect, data, data->items.maxitem - 1);
|
||||
GPU_blend(GPU_BLEND_ALPHA);
|
||||
UI_icon_draw((BLI_rcti_size_x(&rect)) / 2, rect.ymin - 9, ICON_TRIA_DOWN);
|
||||
UI_icon_draw(BLI_rcti_size_x(&rect) / 2, rect.ymin - 9, ICON_TRIA_DOWN);
|
||||
GPU_blend(GPU_BLEND_NONE);
|
||||
}
|
||||
if (data->items.offset) {
|
||||
ui_searchbox_butrect(&rect, data, 0);
|
||||
GPU_blend(GPU_BLEND_ALPHA);
|
||||
UI_icon_draw((BLI_rcti_size_x(&rect)) / 2, rect.ymax - 7, ICON_TRIA_UP);
|
||||
UI_icon_draw(BLI_rcti_size_x(&rect) / 2, rect.ymax - 7, ICON_TRIA_UP);
|
||||
GPU_blend(GPU_BLEND_NONE);
|
||||
}
|
||||
}
|
||||
|
|
|
@ -784,11 +784,10 @@ static void brush_painter_2d_refresh_cache(ImagePaintState *s,
|
|||
|
||||
bool do_random = false;
|
||||
bool do_partial_update = false;
|
||||
bool update_color = ((brush->flag & BRUSH_USE_GRADIENT) &&
|
||||
((ELEM(brush->gradient_stroke_mode,
|
||||
BRUSH_GRADIENT_SPACING_REPEAT,
|
||||
BRUSH_GRADIENT_SPACING_CLAMP)) ||
|
||||
(cache->last_pressure != pressure)));
|
||||
bool update_color = ((brush->flag & BRUSH_USE_GRADIENT) && (ELEM(brush->gradient_stroke_mode,
|
||||
BRUSH_GRADIENT_SPACING_REPEAT,
|
||||
BRUSH_GRADIENT_SPACING_CLAMP) ||
|
||||
(cache->last_pressure != pressure)));
|
||||
float tex_rotation = -brush->mtex.rot;
|
||||
float mask_rotation = -brush->mask_mtex.rot;
|
||||
|
||||
|
|
|
@ -859,7 +859,7 @@ static int graphkeys_box_select_exec(bContext *C, wmOperator *op)
|
|||
* as frame-range one is often used for tweaking timing when "blocking",
|
||||
* while channels is not that useful.
|
||||
*/
|
||||
if ((BLI_rcti_size_x(&rect)) >= (BLI_rcti_size_y(&rect))) {
|
||||
if (BLI_rcti_size_x(&rect) >= BLI_rcti_size_y(&rect)) {
|
||||
mode = BEZT_OK_FRAMERANGE;
|
||||
}
|
||||
else {
|
||||
|
|
|
@ -1013,14 +1013,14 @@ void uiTemplateImageSettings(uiLayout *layout, PointerRNA *imfptr, bool color_ma
|
|||
uiLayoutRow(col, true), imfptr, "color_mode", UI_ITEM_R_EXPAND, IFACE_("Color"), ICON_NONE);
|
||||
|
||||
/* only display depth setting if multiple depths can be used */
|
||||
if ((ELEM(depth_ok,
|
||||
R_IMF_CHAN_DEPTH_1,
|
||||
R_IMF_CHAN_DEPTH_8,
|
||||
R_IMF_CHAN_DEPTH_10,
|
||||
R_IMF_CHAN_DEPTH_12,
|
||||
R_IMF_CHAN_DEPTH_16,
|
||||
R_IMF_CHAN_DEPTH_24,
|
||||
R_IMF_CHAN_DEPTH_32)) == 0) {
|
||||
if (ELEM(depth_ok,
|
||||
R_IMF_CHAN_DEPTH_1,
|
||||
R_IMF_CHAN_DEPTH_8,
|
||||
R_IMF_CHAN_DEPTH_10,
|
||||
R_IMF_CHAN_DEPTH_12,
|
||||
R_IMF_CHAN_DEPTH_16,
|
||||
R_IMF_CHAN_DEPTH_24,
|
||||
R_IMF_CHAN_DEPTH_32) == 0) {
|
||||
uiItemR(uiLayoutRow(col, true), imfptr, "color_depth", UI_ITEM_R_EXPAND, NULL, ICON_NONE);
|
||||
}
|
||||
|
||||
|
|
|
@ -1995,7 +1995,7 @@ static bool image_save_as_draw_check_prop(PointerRNA *ptr,
|
|||
return !(STREQ(prop_id, "filepath") || STREQ(prop_id, "directory") ||
|
||||
STREQ(prop_id, "filename") ||
|
||||
/* when saving a copy, relative path has no effect */
|
||||
((STREQ(prop_id, "relative_path")) && RNA_boolean_get(ptr, "copy")));
|
||||
(STREQ(prop_id, "relative_path") && RNA_boolean_get(ptr, "copy")));
|
||||
}
|
||||
|
||||
static void image_save_as_draw(bContext *UNUSED(C), wmOperator *op)
|
||||
|
|
|
@ -165,9 +165,9 @@ static char txtfmt_lua_format_identifier(const char *str)
|
|||
/* Keep aligned args for readability. */
|
||||
/* clang-format off */
|
||||
|
||||
if ((txtfmt_lua_find_specialvar(str)) != -1) { fmt = FMT_TYPE_SPECIAL;
|
||||
} else if ((txtfmt_lua_find_keyword(str)) != -1) { fmt = FMT_TYPE_KEYWORD;
|
||||
} else { fmt = FMT_TYPE_DEFAULT;
|
||||
if (txtfmt_lua_find_specialvar(str) != -1) { fmt = FMT_TYPE_SPECIAL;
|
||||
} else if (txtfmt_lua_find_keyword(str) != -1) { fmt = FMT_TYPE_KEYWORD;
|
||||
} else { fmt = FMT_TYPE_DEFAULT;
|
||||
}
|
||||
|
||||
/* clang-format on */
|
||||
|
|
|
@ -189,11 +189,11 @@ static char txtfmt_osl_format_identifier(const char *str)
|
|||
/* Keep aligned args for readability. */
|
||||
/* clang-format off */
|
||||
|
||||
if ((txtfmt_osl_find_specialvar(str)) != -1) { fmt = FMT_TYPE_SPECIAL;
|
||||
} else if ((txtfmt_osl_find_builtinfunc(str)) != -1) { fmt = FMT_TYPE_KEYWORD;
|
||||
} else if ((txtfmt_osl_find_reserved(str)) != -1) { fmt = FMT_TYPE_RESERVED;
|
||||
} else if ((txtfmt_osl_find_preprocessor(str)) != -1) { fmt = FMT_TYPE_DIRECTIVE;
|
||||
} else { fmt = FMT_TYPE_DEFAULT;
|
||||
if (txtfmt_osl_find_specialvar(str) != -1) { fmt = FMT_TYPE_SPECIAL;
|
||||
} else if (txtfmt_osl_find_builtinfunc(str) != -1) { fmt = FMT_TYPE_KEYWORD;
|
||||
} else if (txtfmt_osl_find_reserved(str) != -1) { fmt = FMT_TYPE_RESERVED;
|
||||
} else if (txtfmt_osl_find_preprocessor(str) != -1) { fmt = FMT_TYPE_DIRECTIVE;
|
||||
} else { fmt = FMT_TYPE_DEFAULT;
|
||||
}
|
||||
|
||||
/* clang-format on */
|
||||
|
|
|
@ -762,11 +762,11 @@ static char txtfmt_pov_format_identifier(const char *str)
|
|||
/* Keep aligned args for readability. */
|
||||
/* clang-format off */
|
||||
|
||||
if ((txtfmt_pov_find_specialvar(str)) != -1) { fmt = FMT_TYPE_SPECIAL;
|
||||
} else if ((txtfmt_pov_find_keyword(str)) != -1) { fmt = FMT_TYPE_KEYWORD;
|
||||
} else if ((txtfmt_pov_find_reserved_keywords(str)) != -1) { fmt = FMT_TYPE_RESERVED;
|
||||
} else if ((txtfmt_pov_find_reserved_builtins(str)) != -1) { fmt = FMT_TYPE_DIRECTIVE;
|
||||
} else { fmt = FMT_TYPE_DEFAULT;
|
||||
if (txtfmt_pov_find_specialvar(str) != -1) { fmt = FMT_TYPE_SPECIAL;
|
||||
} else if (txtfmt_pov_find_keyword(str) != -1) { fmt = FMT_TYPE_KEYWORD;
|
||||
} else if (txtfmt_pov_find_reserved_keywords(str) != -1) { fmt = FMT_TYPE_RESERVED;
|
||||
} else if (txtfmt_pov_find_reserved_builtins(str) != -1) { fmt = FMT_TYPE_DIRECTIVE;
|
||||
} else { fmt = FMT_TYPE_DEFAULT;
|
||||
}
|
||||
|
||||
/* clang-format on */
|
||||
|
|
|
@ -347,10 +347,10 @@ static int txtfmt_ini_find_bool(const char *string)
|
|||
static char txtfmt_pov_ini_format_identifier(const char *str)
|
||||
{
|
||||
char fmt;
|
||||
if ((txtfmt_ini_find_keyword(str)) != -1) {
|
||||
if (txtfmt_ini_find_keyword(str) != -1) {
|
||||
fmt = FMT_TYPE_KEYWORD;
|
||||
}
|
||||
else if ((txtfmt_ini_find_reserved(str)) != -1) {
|
||||
else if (txtfmt_ini_find_reserved(str) != -1) {
|
||||
fmt = FMT_TYPE_RESERVED;
|
||||
}
|
||||
else {
|
||||
|
|
|
@ -315,10 +315,10 @@ static char txtfmt_py_format_identifier(const char *str)
|
|||
/* Keep aligned args for readability. */
|
||||
/* clang-format off */
|
||||
|
||||
if ((txtfmt_py_find_specialvar(str)) != -1) { fmt = FMT_TYPE_SPECIAL;
|
||||
} else if ((txtfmt_py_find_builtinfunc(str)) != -1) { fmt = FMT_TYPE_KEYWORD;
|
||||
} else if ((txtfmt_py_find_decorator(str)) != -1) { fmt = FMT_TYPE_RESERVED;
|
||||
} else { fmt = FMT_TYPE_DEFAULT;
|
||||
if (txtfmt_py_find_specialvar(str) != -1) { fmt = FMT_TYPE_SPECIAL;
|
||||
} else if (txtfmt_py_find_builtinfunc(str) != -1) { fmt = FMT_TYPE_KEYWORD;
|
||||
} else if (txtfmt_py_find_decorator(str) != -1) { fmt = FMT_TYPE_RESERVED;
|
||||
} else { fmt = FMT_TYPE_DEFAULT;
|
||||
}
|
||||
|
||||
/* clang-format on */
|
||||
|
|
|
@ -78,7 +78,7 @@ static void initSnapSpatial(TransInfo *t, float r_snap[2]);
|
|||
|
||||
bool transdata_check_local_islands(TransInfo *t, short around)
|
||||
{
|
||||
return ((around == V3D_AROUND_LOCAL_ORIGINS) && ((ELEM(t->obedit_type, OB_MESH, OB_GPENCIL))));
|
||||
return ((around == V3D_AROUND_LOCAL_ORIGINS) && (ELEM(t->obedit_type, OB_MESH, OB_GPENCIL)));
|
||||
}
|
||||
|
||||
/* ************************** SPACE DEPENDENT CODE **************************** */
|
||||
|
|
|
@ -695,7 +695,7 @@ int ED_undo_operator_repeat(bContext *C, wmOperator *op)
|
|||
CTX_wm_region_set(C, region_win);
|
||||
}
|
||||
|
||||
if ((WM_operator_repeat_check(C, op)) && (WM_operator_poll(C, op->type)) &&
|
||||
if (WM_operator_repeat_check(C, op) && WM_operator_poll(C, op->type) &&
|
||||
/* NOTE: undo/redo can't run if there are jobs active,
|
||||
* check for screen jobs only so jobs like material/texture/world preview
|
||||
* (which copy their data), won't stop redo, see T29579],
|
||||
|
|
|
@ -1989,7 +1989,7 @@ static StitchState *stitch_init(bContext *C,
|
|||
/* Now, on to generate our uv connectivity data */
|
||||
BM_ITER_MESH (efa, &iter, em->bm, BM_FACES_OF_MESH) {
|
||||
if (!(ts->uv_flag & UV_SYNC_SELECTION) &&
|
||||
((BM_elem_flag_test(efa, BM_ELEM_HIDDEN)) || !BM_elem_flag_test(efa, BM_ELEM_SELECT))) {
|
||||
(BM_elem_flag_test(efa, BM_ELEM_HIDDEN) || !BM_elem_flag_test(efa, BM_ELEM_SELECT))) {
|
||||
continue;
|
||||
}
|
||||
|
||||
|
@ -2172,8 +2172,8 @@ static StitchState *stitch_init(bContext *C,
|
|||
"uv_stitch_selection_stack");
|
||||
|
||||
BM_ITER_MESH (efa, &iter, em->bm, BM_FACES_OF_MESH) {
|
||||
if (!(ts->uv_flag & UV_SYNC_SELECTION) && ((BM_elem_flag_test(efa, BM_ELEM_HIDDEN)) ||
|
||||
!BM_elem_flag_test(efa, BM_ELEM_SELECT))) {
|
||||
if (!(ts->uv_flag & UV_SYNC_SELECTION) &&
|
||||
(BM_elem_flag_test(efa, BM_ELEM_HIDDEN) || !BM_elem_flag_test(efa, BM_ELEM_SELECT))) {
|
||||
continue;
|
||||
}
|
||||
|
||||
|
|
|
@ -315,7 +315,7 @@ static ParamHandle *construct_param_handle(const Scene *scene,
|
|||
|
||||
BM_ITER_MESH_INDEX (efa, &iter, bm, BM_FACES_OF_MESH, i) {
|
||||
|
||||
if ((BM_elem_flag_test(efa, BM_ELEM_HIDDEN)) ||
|
||||
if (BM_elem_flag_test(efa, BM_ELEM_HIDDEN) ||
|
||||
(options->only_selected_faces && BM_elem_flag_test(efa, BM_ELEM_SELECT) == 0)) {
|
||||
continue;
|
||||
}
|
||||
|
@ -404,7 +404,7 @@ static ParamHandle *construct_param_handle_multi(const Scene *scene,
|
|||
|
||||
BM_ITER_MESH_INDEX (efa, &iter, bm, BM_FACES_OF_MESH, i) {
|
||||
|
||||
if ((BM_elem_flag_test(efa, BM_ELEM_HIDDEN)) ||
|
||||
if (BM_elem_flag_test(efa, BM_ELEM_HIDDEN) ||
|
||||
(options->only_selected_faces && BM_elem_flag_test(efa, BM_ELEM_SELECT) == 0)) {
|
||||
continue;
|
||||
}
|
||||
|
|
|
@ -2285,7 +2285,7 @@ struct less_SVertex2D {
|
|||
Vec3r A = x->point2D();
|
||||
Vec3r B = y->point2D();
|
||||
for (unsigned int i = 0; i < 3; i++) {
|
||||
if ((fabs(A[i] - B[i])) < epsilon) {
|
||||
if (fabs(A[i] - B[i]) < epsilon) {
|
||||
continue;
|
||||
}
|
||||
if (A[i] < B[i]) {
|
||||
|
|
|
@ -3442,9 +3442,9 @@ static bool lineart_bounding_area_triangle_intersect(LineartRenderBuffer *fb,
|
|||
return true;
|
||||
}
|
||||
|
||||
if ((lineart_bounding_area_edge_intersect(fb, FBC1, FBC2, ba)) ||
|
||||
(lineart_bounding_area_edge_intersect(fb, FBC2, FBC3, ba)) ||
|
||||
(lineart_bounding_area_edge_intersect(fb, FBC3, FBC1, ba))) {
|
||||
if (lineart_bounding_area_edge_intersect(fb, FBC1, FBC2, ba) ||
|
||||
lineart_bounding_area_edge_intersect(fb, FBC2, FBC3, ba) ||
|
||||
lineart_bounding_area_edge_intersect(fb, FBC3, FBC1, ba)) {
|
||||
return true;
|
||||
}
|
||||
|
||||
|
|
|
@ -88,7 +88,7 @@ static void gpu_node_input_link(GPUNode *node, GPUNodeLink *link, const eGPUType
|
|||
name = outnode->name;
|
||||
input = outnode->inputs.first;
|
||||
|
||||
if ((STR_ELEM(name, "set_value", "set_rgb", "set_rgba")) && (input->type == type)) {
|
||||
if (STR_ELEM(name, "set_value", "set_rgb", "set_rgba") && (input->type == type)) {
|
||||
input = MEM_dupallocN(outnode->inputs.first);
|
||||
if (input->link) {
|
||||
input->link->users++;
|
||||
|
|
|
@ -90,7 +90,7 @@ void GLBackend::platform_init()
|
|||
device |= GPU_DEVICE_INTEL_UHD;
|
||||
}
|
||||
}
|
||||
else if ((strstr(renderer, "Mesa DRI R")) ||
|
||||
else if (strstr(renderer, "Mesa DRI R") ||
|
||||
(strstr(renderer, "Radeon") && strstr(vendor, "X.Org")) ||
|
||||
(strstr(renderer, "AMD") && strstr(vendor, "X.Org")) ||
|
||||
(strstr(renderer, "Gallium ") && strstr(renderer, " on ATI ")) ||
|
||||
|
|
|
@ -1100,7 +1100,7 @@ int rna_show_statusbar_vram_editable(struct PointerRNA *UNUSED(ptr), const char
|
|||
static size_t max_memory_in_megabytes(void)
|
||||
{
|
||||
/* Maximum addressable bytes on this platform. */
|
||||
const size_t limit_bytes = (((size_t)1) << ((sizeof(size_t[8])) - 1));
|
||||
const size_t limit_bytes = (((size_t)1) << (sizeof(size_t[8]) - 1));
|
||||
/* Convert it to megabytes and return. */
|
||||
return (limit_bytes >> 20);
|
||||
}
|
||||
|
|
|
@ -608,7 +608,7 @@ static PyObject *bpy_slot_to_py(BMesh *bm, BMOpSlot *slot)
|
|||
/* keep switch in same order as above */
|
||||
switch (slot->slot_type) {
|
||||
case BMO_OP_SLOT_BOOL:
|
||||
item = PyBool_FromLong((BMO_SLOT_AS_BOOL(slot)));
|
||||
item = PyBool_FromLong(BMO_SLOT_AS_BOOL(slot));
|
||||
break;
|
||||
case BMO_OP_SLOT_INT:
|
||||
item = PyLong_FromLong(BMO_SLOT_AS_INT(slot));
|
||||
|
|
|
@ -189,7 +189,7 @@ static PyObject *bpy_bm_utils_vert_dissolve(PyObject *UNUSED(self), PyObject *ar
|
|||
|
||||
bm = py_vert->bm;
|
||||
|
||||
return PyBool_FromLong((BM_vert_dissolve(bm, py_vert->v)));
|
||||
return PyBool_FromLong(BM_vert_dissolve(bm, py_vert->v));
|
||||
}
|
||||
|
||||
PyDoc_STRVAR(bpy_bm_utils_vert_splice_doc,
|
||||
|
|
|
@ -283,7 +283,7 @@ void BPY_app_handlers_reset(const short do_all)
|
|||
|
||||
for (i = PyList_GET_SIZE(ls) - 1; i >= 0; i--) {
|
||||
|
||||
if ((PyFunction_Check((item = PyList_GET_ITEM(ls, i)))) &&
|
||||
if (PyFunction_Check((item = PyList_GET_ITEM(ls, i))) &&
|
||||
(dict_ptr = _PyObject_GetDictPtr(item)) && (*dict_ptr) &&
|
||||
(PyDict_GetItem(*dict_ptr, perm_id_str) != NULL)) {
|
||||
/* keep */
|
||||
|
|
|
@ -1779,7 +1779,7 @@ static const EnumPropertyItem *enum_items_from_py(PyObject *seq_fast,
|
|||
|
||||
item = seq_fast_items[i];
|
||||
|
||||
if ((PyTuple_CheckExact(item)) && (item_size = PyTuple_GET_SIZE(item)) &&
|
||||
if (PyTuple_CheckExact(item) && (item_size = PyTuple_GET_SIZE(item)) &&
|
||||
(item_size >= 3 && item_size <= 5) &&
|
||||
(tmp.identifier = PyUnicode_AsUTF8AndSize(PyTuple_GET_ITEM(item, 0), &id_str_size)) &&
|
||||
(tmp.name = PyUnicode_AsUTF8AndSize(PyTuple_GET_ITEM(item, 1), &name_str_size)) &&
|
||||
|
|
|
@ -2998,7 +2998,7 @@ static int Matrix_translation_set(MatrixObject *self, PyObject *value, void *UNU
|
|||
return -1;
|
||||
}
|
||||
|
||||
if ((mathutils_array_parse(tvec, 3, 3, value, "Matrix.translation")) == -1) {
|
||||
if (mathutils_array_parse(tvec, 3, 3, value, "Matrix.translation") == -1) {
|
||||
return -1;
|
||||
}
|
||||
|
||||
|
|
Loading…
Reference in New Issue