Cleanup: quiet extra-semicolon warning
authorCampbell Barton <ideasman42@gmail.com>
Sat, 20 Apr 2019 09:28:21 +0000 (11:28 +0200)
committerCampbell Barton <ideasman42@gmail.com>
Sat, 20 Apr 2019 09:30:19 +0000 (11:30 +0200)
15 files changed:
source/blender/blenkernel/BKE_collection.h
source/blender/blenkernel/BKE_node.h
source/blender/blenkernel/intern/particle.c
source/blender/blenkernel/intern/softbody.c
source/blender/blenkernel/intern/studiolight.c
source/blender/depsgraph/intern/builder/deg_builder_nodes.cc
source/blender/draw/intern/draw_manager.h
source/blender/editors/sculpt_paint/paint_image_proj.c
source/blender/gpu/intern/gpu_framebuffer.c
source/blender/makesdna/intern/makesdna.c
source/blender/makesrna/intern/rna_color.c
source/blender/python/mathutils/mathutils.h
source/blender/render/intern/include/texture.h
source/blender/render/intern/source/pipeline.c
source/blender/windowmanager/intern/wm_window.c

index f06d9d1e64e4fbe56b15cf0288f8f26988770f6a..47786629aedef35645de10981da828b261feed9e 100644 (file)
@@ -180,7 +180,8 @@ typedef void (*BKE_scene_collections_Cb)(struct Collection *ob, void *data);
 #define FOREACH_COLLECTION_VISIBLE_OBJECT_RECURSIVE_END \
   } \
   } \
-  }
+  } \
+  ((void)0)
 
 #define FOREACH_COLLECTION_OBJECT_RECURSIVE_BEGIN(_collection, _object) \
   for (Base *_base = (Base *)BKE_collection_object_cache_get(_collection).first; _base; \
index 1f5bb2266d1594f25a81abccc01c12d3d6b82f82..682d0b981fa785c6784f74528a938135b2b8de18 100644 (file)
@@ -491,7 +491,8 @@ const char *nodeStaticSocketInterfaceType(int type, int subtype);
 #define NODE_SOCKET_TYPES_END \
   } \
   BLI_ghashIterator_free(__node_socket_type_iter__); \
-  }
+  } \
+  ((void)0)
 
 struct bNodeSocket *nodeFindSocket(struct bNode *node, int in_out, const char *identifier);
 struct bNodeSocket *nodeAddSocket(struct bNodeTree *ntree,
index 9e121d279baeba261daecf46bf14c99192bba143..5234c2fb0efd312f4ff1d0f05d3d5921da4492d4 100644 (file)
@@ -3143,7 +3143,7 @@ void psys_cache_edit_paths(Depsgraph *depsgraph,
   iter_data.use_weight = use_weight;
 
   if (use_weight) {
-    /* use weight painting colors now... */
+    /* use weight painting colors now... */
   }
   else {
     iter_data.sel_col[0] = (float)edit->sel_col[0] / 255.0f;
index 37b6211916cf6e85103c0aef7d8a8aeaf7e2cf7a..3a7c6527a5060b6766e7bb8f7e0f4821295ece10 100644 (file)
@@ -3515,7 +3515,7 @@ void sbObjectStep(struct Depsgraph *depsgraph,
     return;
   }
   else if (cache_result == PTCACHE_READ_OLD) {
-    ; /* do nothing */
+    /* pass */
   }
   else if (/*ob->id.lib || */ (
       cache->flag &
index df48e505f6ff03e1bf752f59c571b1123a5b42ee..b64f6a1d076582bb8708b950227305f03def2d4c 100644 (file)
@@ -129,12 +129,13 @@ static const char *STUDIOLIGHT_MATCAP_DEFAULT = "basic_1.exr";
 static void studiolight_free(struct StudioLight *sl)
 {
 #define STUDIOLIGHT_DELETE_ICON(s) \
-  { \
+  do { \
     if (s != 0) { \
       BKE_icon_delete(s); \
       s = 0; \
     } \
-  }
+  } while (0)
+
   if (sl->free_function) {
     sl->free_function(sl, sl->free_function_data);
   }
index b37ecb45ac88d41580249bdc90dbb5c99c2edeef..c4a2fc71d1b9c635d745b118a872108a8d616370 100644 (file)
@@ -335,7 +335,7 @@ void DepsgraphNodeBuilder::begin_build()
     entry_tag.name = op_node->name;
     entry_tag.name_tag = op_node->name_tag;
     saved_entry_tags_.push_back(entry_tag);
-  };
+  }
   GSET_FOREACH_END();
 
   /* Make sure graph has no nodes left from previous state. */
index 35e2ab86a8038f49be3d67cd71248f80d95008ab..f9ed5a864dfa053044561d4f1e5890e9100af2a3 100644 (file)
@@ -56,7 +56,9 @@
 
 #  define PROFILE_TIMER_FALLOFF 0.04
 
-#  define PROFILE_START(time_start) double time_start = PIL_check_seconds_timer();
+#  define PROFILE_START(time_start) \
+    double time_start = PIL_check_seconds_timer(); \
+    ((void)0)
 
 #  define PROFILE_END_ACCUM(time_accum, time_start) \
     { \
index 7f8887814a18cb225a586f35b3568b94899b7024..50340210932a5739ff128f4c65598a6e217da5d9 100644 (file)
@@ -395,7 +395,7 @@ typedef struct ProjPaintState {
    * helps as an extra validation step for seam detection. */
   char *faceWindingFlags;
   /** expanded UVs for faces to use as seams. */
-  LoopSeamData(*loopSeamData);
+  LoopSeamData *loopSeamData;
   /** Only needed for when seam_bleed_px is enabled, use to find UV seams. */
   LinkNode **vertFaces;
   /** Seams per vert, to find adjacent seams. */
index 77ecac35835744b98c93eeba4137b1f774525169..dbe1d2c8d69ff86cbe27aaab8095d0a987124ab8 100644 (file)
@@ -562,7 +562,8 @@ bool GPU_framebuffer_check_valid(GPUFrameBuffer *fb, char err_out[256])
 
 #define CHECK_FRAMEBUFFER_IS_BOUND(_fb) \
   BLI_assert(GPU_framebuffer_bound(_fb)); \
-  UNUSED_VARS_NDEBUG(_fb);
+  UNUSED_VARS_NDEBUG(_fb); \
+  ((void)0)
 
 /* Needs to be done after binding. */
 void GPU_framebuffer_viewport_set(GPUFrameBuffer *fb, int x, int y, int w, int h)
index 5564912335e2e33ada09c6a08867d4dd65f9cfab..163165283dd1e50d45cfbbc1ffe39c468d505b23 100644 (file)
@@ -465,7 +465,7 @@ static int add_name(const char *str)
     }
     else if (str[j] == ')') {
       DEBUG_PRINTF(3, "offsetting for brace\n");
-      /* don't get extra offset */
+      /* don't get extra offset */
     }
     else {
       printf("Error during tokening function pointer argument list\n");
index 2e540e8b105b9930d38433df250306e7ebc74dfe..934847fed5014802e68ba89546fda6b5b531cfdf 100644 (file)
@@ -594,9 +594,7 @@ static void rna_ColorManagedColorspaceSettings_reload_update(Main *bmain,
       bool seq_found = false;
 
       if (&scene->sequencer_colorspace_settings != colorspace_settings) {
-        SEQ_BEGIN (scene->ed, seq)
-          ;
-        {
+        SEQ_BEGIN (scene->ed, seq) {
           if (seq->strip && &seq->strip->colorspace_settings == colorspace_settings) {
             seq_found = true;
             break;
@@ -617,9 +615,7 @@ static void rna_ColorManagedColorspaceSettings_reload_update(Main *bmain,
         BKE_sequencer_preprocessed_cache_cleanup_sequence(seq);
       }
       else {
-        SEQ_BEGIN (scene->ed, seq)
-          ;
-        {
+        SEQ_BEGIN (scene->ed, seq) {
           BKE_sequence_free_anim(seq);
         }
         SEQ_END;
index a69d00d10e7812b54bee5ff47471209fca462ad2..8afd60a7324390fe3a53c84e63f047b6fe7349d0 100644 (file)
@@ -32,8 +32,8 @@ extern char BaseMathObject_is_frozen_doc[];
 extern char BaseMathObject_owner_doc[];
 
 #define BASE_MATH_NEW(struct_name, root_type, base_type) \
-  (struct_name *)((base_type ? (base_type)->tp_alloc(base_type, 0) : \
-                               _PyObject_GC_New(&(root_type))));
+  ((struct_name *)((base_type ? (base_type)->tp_alloc(base_type, 0) : \
+                                _PyObject_GC_New(&(root_type)))))
 
 /** BaseMathObject.flag */
 enum {
index d783263c8b8466f0914354c72812bb8f2cee6404..0dd517ed177e0e3bc3106c3fcafe03772b7f16a0 100644 (file)
@@ -31,7 +31,8 @@
       texres->tin = 0.0f; \
     else if (texres->tin > 1.0f) \
       texres->tin = 1.0f; \
-  }
+  } \
+  ((void)0)
 
 #define BRICONTRGB \
   texres->tr = tex->rfac * ((texres->tr - 0.5f) * tex->contrast + tex->bright - 0.5f); \
@@ -58,7 +59,8 @@
       if (texres->tb < 0.0f) \
         texres->tb = 0.0f; \
     } \
-  }
+  } \
+  ((void)0)
 
 struct ImBuf;
 struct Image;
index 091525602ceffdcc8627b9b802341af287f248a0..ef99972aa6aa81e4005e664a9e2a9dc52b9f16ee 100644 (file)
@@ -2864,8 +2864,10 @@ RenderPass *RE_pass_find_by_name(volatile RenderLayer *rl, const char *name, con
 RenderPass *RE_pass_find_by_type(volatile RenderLayer *rl, int passtype, const char *viewname)
 {
 #define CHECK_PASS(NAME) \
-  if (passtype == SCE_PASS_##NAME) \
-    return RE_pass_find_by_name(rl, RE_PASSNAME_##NAME, viewname);
+  if (passtype == SCE_PASS_##NAME) { \
+    return RE_pass_find_by_name(rl, RE_PASSNAME_##NAME, viewname); \
+  } \
+  ((void)0)
 
   CHECK_PASS(COMBINED);
   CHECK_PASS(Z);
index 2fee8dde3f95d6447d658e1fe6ee78d1346fe2ed..2dbe738dd86d79b5cd5964aefae4d38eca5d9738 100644 (file)
@@ -364,6 +364,7 @@ static void wm_block_confirm_quit_cancel(bContext *C, void *arg_block, void *UNU
 }
 
 /** Discard the file changes and quit */
+ATTR_NORETURN
 static void wm_block_confirm_quit_discard(bContext *C, void *arg_block, void *UNUSED(arg))
 {
   wmWindow *win = CTX_wm_window(C);