From 6470056a0d1ed1a07c823d46ea292d69d3c2a8fb Mon Sep 17 00:00:00 2001 From: Campbell Barton Date: Tue, 2 Apr 2019 17:54:04 +1100 Subject: Cleanup: empty expression statement warning --- source/blender/blenkernel/BKE_cloth.h | 12 ++++++------ source/blender/blenkernel/BKE_collection.h | 2 +- source/blender/blenkernel/BKE_layer.h | 12 ++++++------ source/blender/blenkernel/BKE_pbvh.h | 2 +- source/blender/blenkernel/intern/blender.c | 2 +- source/blender/blenkernel/intern/particle.c | 2 +- source/blender/blenkernel/intern/pbvh_bmesh.c | 2 +- source/blender/blenkernel/intern/workspace.c | 2 +- source/blender/blenlib/BLI_listbase.h | 4 ++-- source/blender/blenloader/intern/versioning_280.c | 8 ++++---- source/blender/blenloader/intern/versioning_userdef.c | 2 +- source/blender/depsgraph/DEG_depsgraph_query.h | 2 +- source/blender/draw/engines/eevee/eevee_effects.c | 4 ++-- source/blender/draw/engines/eevee/eevee_lightcache.c | 10 +++++----- source/blender/draw/engines/eevee/eevee_render.c | 2 +- source/blender/draw/engines/gpencil/gpencil_engine.h | 4 ++-- source/blender/draw/intern/DRW_render.h | 4 ++-- source/blender/draw/intern/draw_anim_viz.c | 2 +- source/blender/draw/intern/draw_cache.c | 7 ++++--- source/blender/draw/intern/draw_manager.c | 4 ++-- source/blender/draw/intern/draw_view.c | 2 +- source/blender/draw/modes/edit_curve_mode.c | 4 ++-- source/blender/draw/modes/edit_lattice_mode.c | 2 +- source/blender/draw/modes/object_mode.c | 2 +- .../editors/gizmo_library/gizmo_types/cage2d_gizmo.c | 2 +- source/blender/editors/gpencil/gpencil_paint.c | 4 ++-- source/blender/editors/gpencil/gpencil_primitive.c | 2 +- source/blender/editors/include/ED_anim_api.h | 4 ++-- source/blender/editors/interface/interface_icons.c | 4 ++-- source/blender/editors/mesh/editmesh_utils.c | 2 +- source/blender/editors/object/object_add.c | 2 +- source/blender/editors/object/object_collection.c | 2 +- source/blender/editors/space_outliner/outliner_edit.c | 2 +- source/blender/editors/transform/transform_snap_object.c | 2 +- source/blender/editors/uvedit/uvedit_ops.c | 14 +++++++------- source/blender/gpu/intern/gpu_texture.c | 4 ++-- source/blender/imbuf/intern/iris.c | 8 ++++---- source/blender/makesrna/RNA_access.h | 4 ++-- source/blender/makesrna/intern/rna_armature.c | 2 +- 39 files changed, 79 insertions(+), 78 deletions(-) (limited to 'source/blender') diff --git a/source/blender/blenkernel/BKE_cloth.h b/source/blender/blenkernel/BKE_cloth.h index 6cfce305a04..b5240faed09 100644 --- a/source/blender/blenkernel/BKE_cloth.h +++ b/source/blender/blenkernel/BKE_cloth.h @@ -143,12 +143,12 @@ typedef struct ClothSpring { ClothSpring; // some macro enhancements for vector treatment -#define VECSUBADDSS(v1,v2,aS,v3,bS) {*(v1)-= *(v2)*aS + *(v3)*bS; *(v1+1)-= *(v2+1)*aS + *(v3+1)*bS; *(v1+2)-= *(v2+2)*aS + *(v3+2)*bS;} -#define VECADDSS(v1,v2,aS,v3,bS) {*(v1)= *(v2)*aS + *(v3)*bS; *(v1+1)= *(v2+1)*aS + *(v3+1)*bS; *(v1+2)= *(v2+2)*aS + *(v3+2)*bS;} -#define VECADDS(v1,v2,v3,bS) {*(v1)= *(v2) + *(v3)*bS; *(v1+1)= *(v2+1) + *(v3+1)*bS; *(v1+2)= *(v2+2) + *(v3+2)*bS;} -#define VECSUBMUL(v1,v2,aS) {*(v1)-= *(v2) * aS; *(v1+1)-= *(v2+1) * aS; *(v1+2)-= *(v2+2) * aS;} -#define VECSUBS(v1,v2,v3,bS) {*(v1)= *(v2) - *(v3)*bS; *(v1+1)= *(v2+1) - *(v3+1)*bS; *(v1+2)= *(v2+2) - *(v3+2)*bS;} -#define VECADDMUL(v1,v2,aS) {*(v1)+= *(v2) * aS; *(v1+1)+= *(v2+1) * aS; *(v1+2)+= *(v2+2) * aS;} +#define VECSUBADDSS(v1,v2,aS,v3,bS) {*(v1)-= *(v2)*aS + *(v3)*bS; *(v1+1)-= *(v2+1)*aS + *(v3+1)*bS; *(v1+2)-= *(v2+2)*aS + *(v3+2)*bS;} ((void)0) +#define VECADDSS(v1,v2,aS,v3,bS) {*(v1)= *(v2)*aS + *(v3)*bS; *(v1+1)= *(v2+1)*aS + *(v3+1)*bS; *(v1+2)= *(v2+2)*aS + *(v3+2)*bS;} ((void)0) +#define VECADDS(v1,v2,v3,bS) {*(v1)= *(v2) + *(v3)*bS; *(v1+1)= *(v2+1) + *(v3+1)*bS; *(v1+2)= *(v2+2) + *(v3+2)*bS;} ((void)0) +#define VECSUBMUL(v1,v2,aS) {*(v1)-= *(v2) * aS; *(v1+1)-= *(v2+1) * aS; *(v1+2)-= *(v2+2) * aS;} ((void)0) +#define VECSUBS(v1,v2,v3,bS) {*(v1)= *(v2) - *(v3)*bS; *(v1+1)= *(v2+1) - *(v3+1)*bS; *(v1+2)= *(v2+2) - *(v3+2)*bS;} ((void)0) +#define VECADDMUL(v1,v2,aS) {*(v1)+= *(v2) * aS; *(v1+1)+= *(v2+1) * aS; *(v1+2)+= *(v2+2) * aS;} ((void)0) /* SIMULATION FLAGS: goal flags,.. */ /* These are the bits used in SimSettings.flags. */ diff --git a/source/blender/blenkernel/BKE_collection.h b/source/blender/blenkernel/BKE_collection.h index f46405dceab..180823b791a 100644 --- a/source/blender/blenkernel/BKE_collection.h +++ b/source/blender/blenkernel/BKE_collection.h @@ -203,7 +203,7 @@ void BKE_scene_objects_iterator_end(struct BLI_Iterator *iter); #define FOREACH_COLLECTION_END \ } \ -} +} ((void)0) #define FOREACH_SCENE_OBJECT_BEGIN(scene, _instance) \ ITER_BEGIN(BKE_scene_objects_iterator_begin, \ diff --git a/source/blender/blenkernel/BKE_layer.h b/source/blender/blenkernel/BKE_layer.h index f10f2f44706..8e66aa55b69 100644 --- a/source/blender/blenkernel/BKE_layer.h +++ b/source/blender/blenkernel/BKE_layer.h @@ -344,37 +344,37 @@ bool BKE_view_layer_filter_edit_mesh_has_edges(struct Object *ob, void *user_dat #define BKE_view_layer_array_from_objects_in_edit_mode(view_layer, v3d, r_len) \ BKE_view_layer_array_from_objects_in_mode( \ view_layer, v3d, r_len, { \ - .object_mode = OB_MODE_EDIT}); + .object_mode = OB_MODE_EDIT}) #define BKE_view_layer_array_from_bases_in_edit_mode(view_layer, v3d, r_len) \ BKE_view_layer_array_from_bases_in_mode( \ view_layer, v3d, r_len, { \ - .object_mode = OB_MODE_EDIT}); + .object_mode = OB_MODE_EDIT}) #define BKE_view_layer_array_from_objects_in_edit_mode_unique_data(view_layer, v3d, r_len) \ BKE_view_layer_array_from_objects_in_mode( \ view_layer, v3d, r_len, { \ .object_mode = OB_MODE_EDIT, \ - .no_dup_data = true}); + .no_dup_data = true}) #define BKE_view_layer_array_from_bases_in_edit_mode_unique_data(view_layer, v3d, r_len) \ BKE_view_layer_array_from_bases_in_mode( \ view_layer, v3d, r_len, { \ .object_mode = OB_MODE_EDIT, \ - .no_dup_data = true}); + .no_dup_data = true}) #define BKE_view_layer_array_from_objects_in_edit_mode_unique_data_with_uvs(view_layer, v3d, r_len) \ BKE_view_layer_array_from_objects_in_mode( \ view_layer, v3d, r_len, { \ .object_mode = OB_MODE_EDIT, \ .no_dup_data = true, \ - .filter_fn = BKE_view_layer_filter_edit_mesh_has_uvs}); + .filter_fn = BKE_view_layer_filter_edit_mesh_has_uvs}) #define BKE_view_layer_array_from_objects_in_mode_unique_data(view_layer, v3d, r_len, mode) \ BKE_view_layer_array_from_objects_in_mode( \ view_layer, v3d, r_len, { \ .object_mode = mode, \ - .no_dup_data = true}); + .no_dup_data = true}) #ifdef __cplusplus } diff --git a/source/blender/blenkernel/BKE_pbvh.h b/source/blender/blenkernel/BKE_pbvh.h index 87d1a6c6915..9be2d645fde 100644 --- a/source/blender/blenkernel/BKE_pbvh.h +++ b/source/blender/blenkernel/BKE_pbvh.h @@ -350,7 +350,7 @@ void pbvh_vertex_iter_init(PBVH *bvh, PBVHNode *node, #define BKE_pbvh_vertex_iter_end \ } \ } \ - } + } ((void)0) void BKE_pbvh_node_get_proxies(PBVHNode *node, PBVHProxyNode **proxies, int *proxy_count); void BKE_pbvh_node_free_proxies(PBVHNode *node); diff --git a/source/blender/blenkernel/intern/blender.c b/source/blender/blenkernel/intern/blender.c index 08d49dee754..36e9c8f36dc 100644 --- a/source/blender/blenkernel/intern/blender.c +++ b/source/blender/blenkernel/intern/blender.c @@ -275,7 +275,7 @@ void BKE_blender_userdef_app_template_data_swap(UserDef *userdef_a, UserDef *use memcpy(&(userdef_tmp.id), &(userdef_a->id), sizeof(userdef_tmp.id)); \ memcpy(&(userdef_a->id), &(userdef_b->id), sizeof(userdef_tmp.id)); \ memcpy(&(userdef_b->id), &(userdef_tmp.id), sizeof(userdef_tmp.id)); \ - } + } ((void)0) #define LIST_SWAP(id) { \ SWAP(ListBase, userdef_a->id, userdef_b->id); \ diff --git a/source/blender/blenkernel/intern/particle.c b/source/blender/blenkernel/intern/particle.c index 4abc57ca1c9..35869c44b87 100644 --- a/source/blender/blenkernel/intern/particle.c +++ b/source/blender/blenkernel/intern/particle.c @@ -2373,7 +2373,7 @@ void psys_cache_child_paths( totparent = ctx.totparent; if (editupdate && sim->psys->childcache && totchild == sim->psys->totchildcache) { - ; /* just overwrite the existing cache */ + /* just overwrite the existing cache */ } else { /* clear out old and create new empty path cache */ diff --git a/source/blender/blenkernel/intern/pbvh_bmesh.c b/source/blender/blenkernel/intern/pbvh_bmesh.c index cd97fcf432e..c75a748574c 100644 --- a/source/blender/blenkernel/intern/pbvh_bmesh.c +++ b/source/blender/blenkernel/intern/pbvh_bmesh.c @@ -107,7 +107,7 @@ static void pbvh_bmesh_verify(PBVH *bvh); #define BM_FACES_OF_VERT_ITER_END \ } \ BM_LOOPS_OF_VERT_ITER_END; \ -} +} ((void)0) static void bm_edges_from_tri(BMesh *bm, BMVert *v_tri[3], BMEdge *e_tri[3]) { diff --git a/source/blender/blenkernel/intern/workspace.c b/source/blender/blenkernel/intern/workspace.c index c2bded42bc1..d4adc78f334 100644 --- a/source/blender/blenkernel/intern/workspace.c +++ b/source/blender/blenkernel/intern/workspace.c @@ -331,7 +331,7 @@ WorkSpaceLayout *BKE_workspace_layout_iter_circular( return iter_layout; } } - LISTBASE_CIRCULAR_FORWARD_END(&workspace->layouts, iter_layout, start) + LISTBASE_CIRCULAR_FORWARD_END(&workspace->layouts, iter_layout, start); } return NULL; diff --git a/source/blender/blenlib/BLI_listbase.h b/source/blender/blenlib/BLI_listbase.h index cb0d394bd0b..5e93039c39c 100644 --- a/source/blender/blenlib/BLI_listbase.h +++ b/source/blender/blenlib/BLI_listbase.h @@ -111,7 +111,7 @@ if ((lb)->first && (lb_init || (lb_init = (lb)->first))) { \ #define LISTBASE_CIRCULAR_FORWARD_END(lb, lb_iter, lb_init) \ } while ((lb_iter = (lb_iter)->next ? (lb_iter)->next : (lb)->first), \ (lb_iter != lb_init)); \ -} +} ((void)0) #define LISTBASE_CIRCULAR_BACKWARD_BEGIN(lb, lb_iter, lb_init) \ if ((lb)->last && (lb_init || (lb_init = (lb)->last))) { \ @@ -120,7 +120,7 @@ if ((lb)->last && (lb_init || (lb_init = (lb)->last))) { \ #define LISTBASE_CIRCULAR_BACKWARD_END(lb, lb_iter, lb_init) \ } while ((lb_iter = (lb_iter)->prev ? (lb_iter)->prev : (lb)->last), \ (lb_iter != lb_init)); \ -} +} ((void)0) #define LISTBASE_FOREACH(type, var, list) \ for (type var = (type)((list)->first); \ diff --git a/source/blender/blenloader/intern/versioning_280.c b/source/blender/blenloader/intern/versioning_280.c index 960fb3b417c..4173bb9d99a 100644 --- a/source/blender/blenloader/intern/versioning_280.c +++ b/source/blender/blenloader/intern/versioning_280.c @@ -1434,7 +1434,7 @@ void blo_do_versions_280(FileData *fd, Library *UNUSED(lib), Main *bmain) scene->eevee.flag &= ~_flag; \ } \ } \ - } + } ((void)0) #define EEVEE_GET_INT(_props, _name) \ { \ @@ -1442,7 +1442,7 @@ void blo_do_versions_280(FileData *fd, Library *UNUSED(lib), Main *bmain) if (_idprop != NULL) { \ scene->eevee._name = IDP_Int(_idprop); \ } \ - } + } ((void)0) #define EEVEE_GET_FLOAT(_props, _name) \ { \ @@ -1450,7 +1450,7 @@ void blo_do_versions_280(FileData *fd, Library *UNUSED(lib), Main *bmain) if (_idprop != NULL) { \ scene->eevee._name = IDP_Float(_idprop); \ } \ - } + } ((void)0) #define EEVEE_GET_FLOAT_ARRAY(_props, _name, _length) \ { \ @@ -1461,7 +1461,7 @@ void blo_do_versions_280(FileData *fd, Library *UNUSED(lib), Main *bmain) scene->eevee._name [_i] = _values[_i]; \ } \ } \ - } + } ((void)0) IDProperty *props = IDP_GetPropertyFromGroup(scene->layer_properties, RE_engine_id_BLENDER_EEVEE); EEVEE_GET_BOOL(props, volumetric_enable, SCE_EEVEE_VOLUMETRIC_ENABLED); diff --git a/source/blender/blenloader/intern/versioning_userdef.c b/source/blender/blenloader/intern/versioning_userdef.c index 502b33a2b4a..025bfc55569 100644 --- a/source/blender/blenloader/intern/versioning_userdef.c +++ b/source/blender/blenloader/intern/versioning_userdef.c @@ -52,7 +52,7 @@ static void do_versions_theme(UserDef *userdef, bTheme *btheme) } #define FROM_DEFAULT_V4_UCHAR(member) \ - copy_v4_v4_char(btheme->member, U_theme_default.member); + copy_v4_v4_char(btheme->member, U_theme_default.member) if (!USER_VERSION_ATLEAST(280, 25)) { copy_v4_v4_char(btheme->space_action.anim_preview_range, btheme->space_action.anim_active); diff --git a/source/blender/depsgraph/DEG_depsgraph_query.h b/source/blender/depsgraph/DEG_depsgraph_query.h index 32fc7a301a4..ce4508461ef 100644 --- a/source/blender/depsgraph/DEG_depsgraph_query.h +++ b/source/blender/depsgraph/DEG_depsgraph_query.h @@ -158,7 +158,7 @@ void DEG_iterator_objects_end(struct BLI_Iterator *iter); #define DEG_OBJECT_ITER_END \ ITER_END; \ - } + } ((void)0) /** * Depsgraph objects iterator for draw manager and final render diff --git a/source/blender/draw/engines/eevee/eevee_effects.c b/source/blender/draw/engines/eevee/eevee_effects.c index 9de01bf140b..e276fcd63ce 100644 --- a/source/blender/draw/engines/eevee/eevee_effects.c +++ b/source/blender/draw/engines/eevee/eevee_effects.c @@ -119,14 +119,14 @@ static void eevee_create_shader_downsample(void) GPU_ATTACHMENT_NONE, \ GPU_ATTACHMENT_TEXTURE(tex), \ }); \ -} +} ((void)0) #define CLEANUP_BUFFER(tex, fb, fb_color) { \ /* Cleanup to release memory */ \ DRW_TEXTURE_FREE_SAFE(tex); \ GPU_FRAMEBUFFER_FREE_SAFE(fb); \ GPU_FRAMEBUFFER_FREE_SAFE(fb_color); \ -} +} ((void)0) void EEVEE_effects_init(EEVEE_ViewLayerData *sldata, EEVEE_Data *vedata, Object *camera, const bool minimal) { diff --git a/source/blender/draw/engines/eevee/eevee_lightcache.c b/source/blender/draw/engines/eevee/eevee_lightcache.c index d5c322356d8..812af1c6ac0 100644 --- a/source/blender/draw/engines/eevee/eevee_lightcache.c +++ b/source/blender/draw/engines/eevee/eevee_lightcache.c @@ -984,15 +984,15 @@ static bool eevee_lightbake_cube_comp(EEVEE_LightProbe *prb_a, EEVEE_LightProbe bool sorted = false; \ while (!sorted) { \ sorted = true; \ - for (int i = 0; i < (elems_len) - 1; ++i) { \ - if ((comp_fn)((elems) + i, (elems) + i+1)) { \ - SWAP(elems_type, (elems)[i], (elems)[i+1]); \ - SWAP(LightProbe *, (prbs)[i], (prbs)[i+1]); \ + for (int i = 0; i < (elems_len) - 1; i++) { \ + if ((comp_fn)((elems) + i, (elems) + i + 1)) { \ + SWAP(elems_type, (elems)[i], (elems)[i + 1]); \ + SWAP(LightProbe *, (prbs)[i], (prbs)[i + 1]); \ sorted = false; \ } \ } \ } \ -} +} ((void)0) static void eevee_lightbake_gather_probes(EEVEE_LightBake *lbake) { diff --git a/source/blender/draw/engines/eevee/eevee_render.c b/source/blender/draw/engines/eevee/eevee_render.c index 07be0ad750d..e563fa8b9ce 100644 --- a/source/blender/draw/engines/eevee/eevee_render.c +++ b/source/blender/draw/engines/eevee/eevee_render.c @@ -616,7 +616,7 @@ void EEVEE_render_update_passes(RenderEngine *engine, Scene *scene, ViewLayer *v else if (channels == 3) type = SOCK_VECTOR; \ else type = SOCK_FLOAT; \ RE_engine_register_pass(engine, scene, view_layer, RE_PASSNAME_ ## name, channels, chanid, type); \ - } + } ((void)0) CHECK_PASS(Z, 1, "Z"); CHECK_PASS(MIST, 1, "Z"); diff --git a/source/blender/draw/engines/gpencil/gpencil_engine.h b/source/blender/draw/engines/gpencil/gpencil_engine.h index 860dd728805..8af74ec7451 100644 --- a/source/blender/draw/engines/gpencil/gpencil_engine.h +++ b/source/blender/draw/engines/gpencil/gpencil_engine.h @@ -459,7 +459,7 @@ void GPENCIL_render_to_image(void *vedata, struct RenderEngine *engine, struct R GPU_framebuffer_clear_color_depth(fbl->multisample_fb, (const float[4]){0.0f}, 1.0f); \ DRW_stats_query_end(); \ } \ -} +} ((void)0) #define MULTISAMPLE_GP_SYNC_DISABLE(lvl, fbl, fb, txl) { \ if ((lvl > 0) && (fbl->multisample_fb != NULL)) { \ @@ -468,7 +468,7 @@ void GPENCIL_render_to_image(void *vedata, struct RenderEngine *engine, struct R DRW_multisamples_resolve(txl->multisample_depth, txl->multisample_color, true); \ DRW_stats_query_end(); \ } \ -} +} ((void)0) #define GPENCIL_3D_DRAWMODE(ob, gpd) \ ((gpd) && (gpd->draw_mode == GP_DRAWMODE_3D) && \ diff --git a/source/blender/draw/intern/DRW_render.h b/source/blender/draw/intern/DRW_render.h index 5c505f1b234..3d4618021e8 100644 --- a/source/blender/draw/intern/DRW_render.h +++ b/source/blender/draw/intern/DRW_render.h @@ -120,7 +120,7 @@ typedef char DRWViewportEmptyList; DRW_multisamples_resolve(dtxl->multisample_depth, dtxl->multisample_color, true); \ DRW_stats_query_end(); \ } \ -} +} ((void)0) #define MULTISAMPLE_SYNC_DISABLE_NO_DEPTH(dfbl, dtxl) { \ if (dfbl->multisample_fb != NULL) { \ @@ -129,7 +129,7 @@ typedef char DRWViewportEmptyList; DRW_multisamples_resolve(dtxl->multisample_depth, dtxl->multisample_color, false); \ DRW_stats_query_end(); \ } \ -} +} ((void)0) diff --git a/source/blender/draw/intern/draw_anim_viz.c b/source/blender/draw/intern/draw_anim_viz.c index 09625b9b47e..03566de01d2 100644 --- a/source/blender/draw/intern/draw_anim_viz.c +++ b/source/blender/draw/intern/draw_anim_viz.c @@ -309,7 +309,7 @@ static void MPATH_draw_scene(void *vedata) DRW_draw_pass(psl->lines); DRW_draw_pass(psl->points); - MULTISAMPLE_SYNC_DISABLE_NO_DEPTH(dfbl, dtxl) + MULTISAMPLE_SYNC_DISABLE_NO_DEPTH(dfbl, dtxl); } /* *************************** Draw Engine Defines ****************************** */ diff --git a/source/blender/draw/intern/draw_cache.c b/source/blender/draw/intern/draw_cache.c index cdbbcf9ec2b..f369048ec2f 100644 --- a/source/blender/draw/intern/draw_cache.c +++ b/source/blender/draw/intern/draw_cache.c @@ -2751,9 +2751,10 @@ static const float staticSine[16] = { 0.994521895368f, 1.0f, }; -#define set_vert(a, b, quarter) \ - copy_v2_fl2(pos, (quarter % 2 == 0) ? -(a) : (a), (quarter < 2) ? -(b) : (b)); \ - GPU_vertbuf_attr_set(vbo, attr_id.pos, v++, pos); +#define set_vert(a, b, quarter) { \ + copy_v2_fl2(pos, (quarter % 2 == 0) ? -(a) : (a), (quarter < 2) ? -(b) : (b)); \ + GPU_vertbuf_attr_set(vbo, attr_id.pos, v++, pos); \ + } ((void)0) GPUBatch *DRW_cache_bone_dof_sphere_get(void) { diff --git a/source/blender/draw/intern/draw_manager.c b/source/blender/draw/intern/draw_manager.c index 44ec9926d59..66b5ec22af9 100644 --- a/source/blender/draw/intern/draw_manager.c +++ b/source/blender/draw/intern/draw_manager.c @@ -1716,7 +1716,7 @@ bool DRW_render_check_grease_pencil(Depsgraph *depsgraph) } } } - DEG_OBJECT_ITER_FOR_RENDER_ENGINE_END + DEG_OBJECT_ITER_FOR_RENDER_ENGINE_END; return false; } @@ -1959,7 +1959,7 @@ void DRW_render_object_iter( drw_batch_cache_generate_requested(ob); } } - DEG_OBJECT_ITER_END + DEG_OBJECT_ITER_END; } /* Assume a valid gl context is bound (and that the gl_context_mutex has been acquired). diff --git a/source/blender/draw/intern/draw_view.c b/source/blender/draw/intern/draw_view.c index 71bf37b4e7a..b907452dad2 100644 --- a/source/blender/draw/intern/draw_view.c +++ b/source/blender/draw/intern/draw_view.c @@ -247,7 +247,7 @@ void DRW_draw_cursor(void) #define CURSOR_EDGE(axis_vec, axis, sign) { \ CURSOR_VERT(axis_vec, axis, sign 1.0f); \ CURSOR_VERT(axis_vec, axis, sign 0.25f); \ - } + } ((void)0) for (int axis = 0; axis < 3; axis++) { float axis_vec[3] = {0}; diff --git a/source/blender/draw/modes/edit_curve_mode.c b/source/blender/draw/modes/edit_curve_mode.c index 8c6ea162adc..a1741c7e967 100644 --- a/source/blender/draw/modes/edit_curve_mode.c +++ b/source/blender/draw/modes/edit_curve_mode.c @@ -305,7 +305,7 @@ static void EDIT_CURVE_draw_scene(void *vedata) DRW_draw_pass(psl->wire_pass); - MULTISAMPLE_SYNC_DISABLE(dfbl, dtxl) + MULTISAMPLE_SYNC_DISABLE(dfbl, dtxl); } /* Unfortunately this pass cannot be AA'd without @@ -318,7 +318,7 @@ static void EDIT_CURVE_draw_scene(void *vedata) DRW_draw_pass(psl->wire_pass_xray); - MULTISAMPLE_SYNC_DISABLE_NO_DEPTH(dfbl, dtxl) + MULTISAMPLE_SYNC_DISABLE_NO_DEPTH(dfbl, dtxl); } /* Thoses passes don't write to depth and are AA'ed using other tricks. */ diff --git a/source/blender/draw/modes/edit_lattice_mode.c b/source/blender/draw/modes/edit_lattice_mode.c index 9ea76deed3a..3db167fc1fd 100644 --- a/source/blender/draw/modes/edit_lattice_mode.c +++ b/source/blender/draw/modes/edit_lattice_mode.c @@ -262,7 +262,7 @@ static void EDIT_LATTICE_draw_scene(void *vedata) DRW_draw_pass(psl->wire_pass); DRW_draw_pass(psl->vert_pass); - MULTISAMPLE_SYNC_DISABLE(dfbl, dtxl) + MULTISAMPLE_SYNC_DISABLE(dfbl, dtxl); /* If you changed framebuffer, double check you rebind * the default one with its textures attached before finishing */ diff --git a/source/blender/draw/modes/object_mode.c b/source/blender/draw/modes/object_mode.c index 3cb3d28a29c..563d1267885 100644 --- a/source/blender/draw/modes/object_mode.c +++ b/source/blender/draw/modes/object_mode.c @@ -3308,7 +3308,7 @@ static void OBJECT_draw_scene(void *vedata) DRW_draw_pass(psl->particle); DRW_draw_pass(stl->g_data->sgl.bone_axes); - MULTISAMPLE_SYNC_DISABLE(dfbl, dtxl) + MULTISAMPLE_SYNC_DISABLE(dfbl, dtxl); DRW_draw_pass(stl->g_data->sgl.image_empties); diff --git a/source/blender/editors/gizmo_library/gizmo_types/cage2d_gizmo.c b/source/blender/editors/gizmo_library/gizmo_types/cage2d_gizmo.c index f4f4c6425c2..45b7b37eb8d 100644 --- a/source/blender/editors/gizmo_library/gizmo_types/cage2d_gizmo.c +++ b/source/blender/editors/gizmo_library/gizmo_types/cage2d_gizmo.c @@ -759,7 +759,7 @@ static int gizmo_cage2d_test_select( r.ymin = -size[1] + margin[1]; r.xmax = size[0] - margin[0]; r.ymax = size[1] - margin[1]; - }; + } bool isect = BLI_rctf_isect_pt_v(&r, point_local); if (isect) { return ED_GIZMO_CAGE2D_PART_TRANSLATE; diff --git a/source/blender/editors/gpencil/gpencil_paint.c b/source/blender/editors/gpencil/gpencil_paint.c index cb58542dc38..df40422e1c1 100644 --- a/source/blender/editors/gpencil/gpencil_paint.c +++ b/source/blender/editors/gpencil/gpencil_paint.c @@ -509,11 +509,11 @@ static void gp_brush_jitter( static void gp_brush_angle(bGPdata *gpd, Brush *brush, tGPspoint *pt, const float mval[2]) { float mvec[2]; - float sen = brush->gpencil_settings->draw_angle_factor; /* sensitivity */; + float sen = brush->gpencil_settings->draw_angle_factor; /* sensitivity */ float fac; float mpressure; - /* default angle of brush in radians */; + /* default angle of brush in radians */ float angle = brush->gpencil_settings->draw_angle; /* angle vector of the brush with full thickness */ float v0[2] = { cos(angle), sin(angle) }; diff --git a/source/blender/editors/gpencil/gpencil_primitive.c b/source/blender/editors/gpencil/gpencil_primitive.c index bde6bd30b76..f6571773706 100644 --- a/source/blender/editors/gpencil/gpencil_primitive.c +++ b/source/blender/editors/gpencil/gpencil_primitive.c @@ -834,7 +834,7 @@ static void gp_primitive_update_strokes(bContext *C, tGPDprimitive *tgpi) const float fac = p2d->rnd[0] * exfactor * jitter; /* vector */ - float mvec[2], svec[2];; + float mvec[2], svec[2]; if (i > 0) { mvec[0] = (p2d->x - (p2d - 1)->x); mvec[1] = (p2d->y - (p2d - 1)->y); diff --git a/source/blender/editors/include/ED_anim_api.h b/source/blender/editors/include/ED_anim_api.h index 99b634ddfcc..6dccc55d3ff 100644 --- a/source/blender/editors/include/ED_anim_api.h +++ b/source/blender/editors/include/ED_anim_api.h @@ -735,7 +735,7 @@ float ANIM_unit_mapping_get_factor(struct Scene *scene, struct ID *id, struct FC if (smode == ACHANNEL_SETFLAG_INVERT) (channel)->flag ^= (sflag); \ else if (smode == ACHANNEL_SETFLAG_ADD) (channel)->flag |= (sflag); \ else (channel)->flag &= ~(sflag); \ - } + } ((void)0) /* set/clear/toggle macro, where the flag is negative * - channel - channel with a 'flag' member that we're setting @@ -747,7 +747,7 @@ float ANIM_unit_mapping_get_factor(struct Scene *scene, struct ID *id, struct FC if (smode == ACHANNEL_SETFLAG_INVERT) (channel)->flag ^= (sflag); \ else if (smode == ACHANNEL_SETFLAG_ADD) (channel)->flag &= ~(sflag); \ else (channel)->flag |= (sflag); \ - } + } ((void)0) /* --------- anim_deps.c, animation updates -------- */ diff --git a/source/blender/editors/interface/interface_icons.c b/source/blender/editors/interface/interface_icons.c index 96086b21ab2..dcfeaf64699 100644 --- a/source/blender/editors/interface/interface_icons.c +++ b/source/blender/editors/interface/interface_icons.c @@ -476,7 +476,7 @@ static void init_brush_icons(void) di = def_internal_icon(NULL, icon_id, 0, 0, w, ICON_TYPE_BUFFER, 0); \ di->data.buffer.image->datatoc_rect = rect; \ di->data.buffer.image->datatoc_size = size; \ - } + } ((void)0) /* end INIT_BRUSH_ICON */ const int w = 96; /* warning, brush size hardcoded in C, but it gets scaled */ @@ -614,7 +614,7 @@ static void init_event_icons(void) di->data.input.icon = icon_id; \ di->data.input.next = di_next; \ di_next = di; \ - } + } ((void)0) /* end INIT_EVENT_ICON */ const int w = 16; /* DUMMY */ diff --git a/source/blender/editors/mesh/editmesh_utils.c b/source/blender/editors/mesh/editmesh_utils.c index 68a2f382278..3a4f9d461ec 100644 --- a/source/blender/editors/mesh/editmesh_utils.c +++ b/source/blender/editors/mesh/editmesh_utils.c @@ -1039,7 +1039,7 @@ void EDBM_verts_mirror_cache_begin_ex( BLI_kdtree_3d_balance(tree); } -#define VERT_INTPTR(_v, _i) r_index ? &r_index[_i] : BM_ELEM_CD_GET_VOID_P(_v, cd_vmirr_offset); +#define VERT_INTPTR(_v, _i) (r_index ? &r_index[_i] : BM_ELEM_CD_GET_VOID_P(_v, cd_vmirr_offset)) BM_ITER_MESH_INDEX (v, &iter, bm, BM_VERTS_OF_MESH, i) { BLI_assert(BM_elem_index_get(v) == i); diff --git a/source/blender/editors/object/object_add.c b/source/blender/editors/object/object_add.c index 42c5c2b3576..9269c25f54b 100644 --- a/source/blender/editors/object/object_add.c +++ b/source/blender/editors/object/object_add.c @@ -2215,7 +2215,7 @@ static Base *object_add_duplicate_internal(Main *bmain, Scene *scene, ViewLayer Object *obn; if (ob->mode & OB_MODE_POSE) { - ; /* nothing? */ + /* nothing? */ } else { obn = ID_NEW_SET(ob, BKE_object_duplicate(bmain, ob, dupflag)); diff --git a/source/blender/editors/object/object_collection.c b/source/blender/editors/object/object_collection.c index c226ad7a47f..e8552c823bf 100644 --- a/source/blender/editors/object/object_collection.c +++ b/source/blender/editors/object/object_collection.c @@ -320,7 +320,7 @@ static int collection_objects_remove_exec(bContext *C, wmOperator *op) } CTX_DATA_END; } - FOREACH_COLLECTION_END + FOREACH_COLLECTION_END; if (!updated) return OPERATOR_CANCELLED; diff --git a/source/blender/editors/space_outliner/outliner_edit.c b/source/blender/editors/space_outliner/outliner_edit.c index aa9c462d74e..8b8dff9dc27 100644 --- a/source/blender/editors/space_outliner/outliner_edit.c +++ b/source/blender/editors/space_outliner/outliner_edit.c @@ -254,7 +254,7 @@ static void do_item_rename(ARegion *ar, TreeElement *te, TreeStoreElem *tselem, /* can't rename rna datablocks entries or listbases */ if (ELEM(tselem->type, TSE_RNA_STRUCT, TSE_RNA_PROPERTY, TSE_RNA_ARRAY_ELEM, TSE_ID_BASE, TSE_SCENE_OBJECTS_BASE)) { - /* do nothing */; + /* do nothing */ } else if (ELEM(tselem->type, TSE_ANIM_DATA, TSE_NLA, TSE_DEFGROUP_BASE, TSE_CONSTRAINT_BASE, TSE_MODIFIER_BASE, TSE_DRIVER_BASE, TSE_POSE_BASE, TSE_POSEGRP_BASE, TSE_R_LAYER_BASE, TSE_SCENE_COLLECTION_BASE, diff --git a/source/blender/editors/transform/transform_snap_object.c b/source/blender/editors/transform/transform_snap_object.c index 62ecfdbc59d..ea3657b8729 100644 --- a/source/blender/editors/transform/transform_snap_object.c +++ b/source/blender/editors/transform/transform_snap_object.c @@ -1327,7 +1327,7 @@ static short snap_mesh_edge_verts_mixed( neasrest_precalc.ray_direction, v_pair[0], v_pair[1], &lambda)) { - /* do nothing */; + /* do nothing */ } else if (lambda < 0.25f || 0.75f < lambda) { int v_id = lambda < 0.5f ? 0 : 1; diff --git a/source/blender/editors/uvedit/uvedit_ops.c b/source/blender/editors/uvedit/uvedit_ops.c index b8c2c461f9b..a4247ec370d 100644 --- a/source/blender/editors/uvedit/uvedit_ops.c +++ b/source/blender/editors/uvedit/uvedit_ops.c @@ -2627,7 +2627,7 @@ static void UV_OT_select(wmOperatorType *ot) /* api callbacks */ ot->exec = uv_select_exec; ot->invoke = uv_select_invoke; - ot->poll = ED_operator_uvedit; /* requires space image */; + ot->poll = ED_operator_uvedit; /* requires space image */ /* properties */ RNA_def_boolean(ot->srna, "extend", 0, @@ -2676,7 +2676,7 @@ static void UV_OT_select_loop(wmOperatorType *ot) /* api callbacks */ ot->exec = uv_select_loop_exec; ot->invoke = uv_select_loop_invoke; - ot->poll = ED_operator_uvedit; /* requires space image */; + ot->poll = ED_operator_uvedit; /* requires space image */ /* properties */ RNA_def_boolean(ot->srna, "extend", 0, @@ -2811,7 +2811,7 @@ static void UV_OT_select_linked_pick(wmOperatorType *ot) /* api callbacks */ ot->invoke = uv_select_linked_pick_invoke; ot->exec = uv_select_linked_pick_exec; - ot->poll = ED_operator_uvedit; /* requires space image */; + ot->poll = ED_operator_uvedit; /* requires space image */ /* properties */ RNA_def_boolean(ot->srna, "extend", 0, @@ -2917,7 +2917,7 @@ static void UV_OT_select_split(wmOperatorType *ot) /* api callbacks */ ot->exec = uv_select_split_exec; - ot->poll = ED_operator_uvedit; /* requires space image */; + ot->poll = ED_operator_uvedit; /* requires space image */ } static void uv_select_sync_flush(ToolSettings *ts, BMEditMesh *em, const short select) @@ -3312,7 +3312,7 @@ static void UV_OT_select_box(wmOperatorType *ot) ot->invoke = WM_gesture_box_invoke; ot->exec = uv_box_select_exec; ot->modal = WM_gesture_box_modal; - ot->poll = ED_operator_uvedit_space_image; /* requires space image */; + ot->poll = ED_operator_uvedit_space_image; /* requires space image */ ot->cancel = WM_gesture_box_cancel; /* flags */ @@ -3460,7 +3460,7 @@ static void UV_OT_select_circle(wmOperatorType *ot) ot->invoke = WM_gesture_circle_invoke; ot->modal = WM_gesture_circle_modal; ot->exec = uv_circle_select_exec; - ot->poll = ED_operator_uvedit_space_image; /* requires space image */; + ot->poll = ED_operator_uvedit_space_image; /* requires space image */ ot->cancel = WM_gesture_circle_cancel; /* flags */ @@ -3697,7 +3697,7 @@ static void UV_OT_snap_cursor(wmOperatorType *ot) /* api callbacks */ ot->exec = uv_snap_cursor_exec; - ot->poll = ED_operator_uvedit_space_image; /* requires space image */; + ot->poll = ED_operator_uvedit_space_image; /* requires space image */ /* properties */ RNA_def_enum(ot->srna, "target", target_items, 0, "Target", "Target to snap the selected UVs to"); diff --git a/source/blender/gpu/intern/gpu_texture.c b/source/blender/gpu/intern/gpu_texture.c index 5398db82586..8aafe349be8 100644 --- a/source/blender/gpu/intern/gpu_texture.c +++ b/source/blender/gpu/intern/gpu_texture.c @@ -1385,13 +1385,13 @@ int GPU_texture_bound_number(GPUTexture *tex) return tex->number; } -#define WARN_NOT_BOUND(_tex) do { \ +#define WARN_NOT_BOUND(_tex) { \ if (_tex->number == -1) { \ fprintf(stderr, "Warning : Trying to set parameter on a texture not bound.\n"); \ BLI_assert(0); \ return; \ } \ -} while (0); +} ((void)0) void GPU_texture_generate_mipmap(GPUTexture *tex) { diff --git a/source/blender/imbuf/intern/iris.c b/source/blender/imbuf/intern/iris.c index cc6d99e2a90..5bed5b28b14 100644 --- a/source/blender/imbuf/intern/iris.c +++ b/source/blender/imbuf/intern/iris.c @@ -602,10 +602,10 @@ static int expandrow2( float pixel_f; #define EXPAND_CAPACITY_AT_INPUT_OK_OR_FAIL(iptr_next) \ - if (UNLIKELY(iptr_next > iptr_end)) { goto fail; } + if (UNLIKELY(iptr_next > iptr_end)) { goto fail; } ((void)0) #define EXPAND_CAPACITY_AT_OUTPUT_OK_OR_FAIL(optr_next) \ - if (UNLIKELY(optr_next > optr_end)) { goto fail; } + if (UNLIKELY(optr_next > optr_end)) { goto fail; } ((void)0) optr += z; optr_end += z; @@ -683,10 +683,10 @@ static int expandrow( uchar pixel, count; #define EXPAND_CAPACITY_AT_INPUT_OK_OR_FAIL(iptr_next) \ - if (UNLIKELY(iptr_next > iptr_end)) { goto fail; } + if (UNLIKELY(iptr_next > iptr_end)) { goto fail; } ((void)0) #define EXPAND_CAPACITY_AT_OUTPUT_OK_OR_FAIL(optr_next) \ - if (UNLIKELY(optr_next > optr_end)) { goto fail; } + if (UNLIKELY(optr_next > optr_end)) { goto fail; } ((void)0) optr += z; optr_end += z; diff --git a/source/blender/makesrna/RNA_access.h b/source/blender/makesrna/RNA_access.h index 5866302a852..70194d71e26 100644 --- a/source/blender/makesrna/RNA_access.h +++ b/source/blender/makesrna/RNA_access.h @@ -1157,7 +1157,7 @@ void RNA_collection_clear(PointerRNA *ptr, const char *name); #define RNA_PROP_END \ } \ RNA_property_collection_end(&rna_macro_iter); \ - } + } ((void)0) #define RNA_STRUCT_BEGIN(sptr, prop) \ { \ @@ -1174,7 +1174,7 @@ void RNA_collection_clear(PointerRNA *ptr, const char *name); #define RNA_STRUCT_END \ } \ RNA_property_collection_end(&rna_macro_iter); \ - } + } ((void)0) /* check if the idproperty exists, for operators */ bool RNA_property_is_set_ex(PointerRNA *ptr, PropertyRNA *prop, bool use_ghost); diff --git a/source/blender/makesrna/intern/rna_armature.c b/source/blender/makesrna/intern/rna_armature.c index 1c16abcd12f..4440cd2313c 100644 --- a/source/blender/makesrna/intern/rna_armature.c +++ b/source/blender/makesrna/intern/rna_armature.c @@ -580,7 +580,7 @@ void rna_def_bone_curved_common(StructRNA *srna, bool is_posebone) RNA_def_property_update(prop, NC_OBJECT | ND_POSE, "rna_Pose_update"); \ else \ RNA_def_property_update(prop, 0, "rna_Armature_update_data"); \ - } (void)0; + } ((void)0) PropertyRNA *prop; -- cgit v1.2.3