Cleanup: fix compiler warnings
authorBrecht Van Lommel <brechtvanlommel@gmail.com>
Sun, 26 Jan 2020 15:35:58 +0000 (16:35 +0100)
committerBrecht Van Lommel <brechtvanlommel@gmail.com>
Mon, 27 Jan 2020 11:19:49 +0000 (12:19 +0100)
12 files changed:
CMakeLists.txt
extern/quadriflow/3rd/lemon-1.3.1/lemon/random.h
extern/quadriflow/patches/blender.patch
intern/opensubdiv/CMakeLists.txt
intern/openvdb/openvdb_util.cc
source/blender/blenkernel/intern/seqprefetch.c
source/blender/blenkernel/intern/shrinkwrap.c
source/blender/editors/gpencil/gpencil_primitive.c
source/blender/editors/object/object_remesh.c
source/blender/editors/sculpt_paint/sculpt.c
source/blender/usd/intern/abstract_hierarchy_iterator.cc
source/blender/windowmanager/message_bus/intern/wm_message_bus_rna.c

index 70196d00df6c5c9ee23a2603fc0dce6df4c801a1..bbb528607c8d53d63388ab8ff0270b08b4d0e608 100644 (file)
@@ -97,6 +97,9 @@ cmake_policy(SET CMP0010 NEW)
 # Input directories must have CMakeLists.txt
 cmake_policy(SET CMP0014 NEW)
 
+# Silence draco warning on macOS, new policy works fine.
+cmake_policy(SET CMP0068 NEW)
+
 #-----------------------------------------------------------------------------
 # Load some macros.
 include(build_files/cmake/macros.cmake)
index 8de74ede8a9e209a81d5f0c0d126f7d90fd27950..f9861f3916996ba5e96d30f62e3c398944d2c8b8 100644 (file)
@@ -249,8 +249,8 @@ namespace lemon {
 
         current = state + length;
 
-        register Word *curr = state + length - 1;
-        register long num;
+        Word *curr = state + length - 1;
+        long num;
 
         num = length - shift;
         while (num--) {
index 55fdd123e3bab63d72c167e13008cdea3aacd4d9..eceda4893933f2041aa4da429e70ea1fca30dee5 100644 (file)
@@ -116,3 +116,18 @@ index c333256..8cc41da 100644
  
          for (uint32_t i = 0; i != mV[l + 1].cols(); ++i) {
              Vector2i upper = toUpper.col(i);
+diff --git a/extern/quadriflow/3rd/lemon-1.3.1/lemon/random.h b/extern/quadriflow/3rd/lemon-1.3.1/lemon/random.h
+index 8de74ede8a9..f9861f39169 100644
+--- a/extern/quadriflow/3rd/lemon-1.3.1/lemon/random.h
++++ b/extern/quadriflow/3rd/lemon-1.3.1/lemon/random.h
+@@ -249,8 +249,8 @@ namespace lemon {
+         current = state + length;
+-        register Word *curr = state + length - 1;
+-        register long num;
++        Word *curr = state + length - 1;
++        long num;
+         num = length - shift;
+         while (num--) {
index 268b4e31d7dcf2307bb057105300dff42612f7f3..e7292872e9cec391a7ad4027674fc583ef94d9c9 100644 (file)
@@ -106,6 +106,7 @@ if(WITH_OPENSUBDIV)
   data_to_c_simple(shader/gpu_shader_opensubdiv_fragment.glsl SRC)
 
   add_definitions(-DGLEW_STATIC)
+  add_definitions(-DOSD_USES_GLEW)
 
   if(WIN32)
     add_definitions(-DNOMINMAX)
index a221a537851705912cdc5e5908449a62327e8a48..899b41ff09bb9dc50241e42753efed94c1dda324 100644 (file)
@@ -27,5 +27,10 @@ ScopeTimer::ScopeTimer(const std::string &message) : m_message(message), m_timer
 
 ScopeTimer::~ScopeTimer()
 {
-  std::printf("%s: %fms\n", m_message.c_str(), m_timer.delta());
+#if OPENVDB_LIBRARY_MAJOR_VERSION_NUMBER >= 7
+  double delta = m_timer.milliseconds();
+#else
+  double delta = m_timer.delta(); /* Deprecated in OpenVDB 7. */
+#endif
+  std::printf("%s: %fms\n", m_message.c_str(), delta);
 }
index 6dd1c47407f1d759b364fe27457b4f799607fed6..8c9097e1d4e307b7a321d6640bc5e4a2c83d768e 100644 (file)
@@ -416,9 +416,7 @@ static PrefetchJob *seq_prefetch_start(const SeqRenderData *context, float cfra)
   pfjob->stop = false;
   pfjob->running = true;
 
-  if (&pfjob->threads) {
-    BLI_threadpool_remove(&pfjob->threads, pfjob);
-  }
+  BLI_threadpool_remove(&pfjob->threads, pfjob);
   BLI_threadpool_insert(&pfjob->threads, pfjob);
 
   return pfjob;
index a2699e767e98fe8a7cf5743900176c98feb2f0e5..49a295c6a9e559accac0030cd07f2aad881bffe7 100644 (file)
@@ -1511,7 +1511,7 @@ void BKE_shrinkwrap_mesh_nearest_surface_deform(struct bContext *C,
 {
   Depsgraph *depsgraph = CTX_data_depsgraph_pointer(C);
   struct Scene *sce = CTX_data_scene(C);
-  ShrinkwrapModifierData ssmd = {0};
+  ShrinkwrapModifierData ssmd = {{0}};
   ModifierEvalContext ctx = {depsgraph, ob_source, 0};
   int totvert;
 
@@ -1532,7 +1532,7 @@ void BKE_shrinkwrap_mesh_nearest_surface_deform(struct bContext *C,
 
 void BKE_shrinkwrap_remesh_target_project(Mesh *src_me, Mesh *target_me, Object *ob_target)
 {
-  ShrinkwrapModifierData ssmd = {0};
+  ShrinkwrapModifierData ssmd = {{0}};
   int totvert;
 
   ssmd.target = ob_target;
index f0ff38e60b1112699e17e6b6f56934c87afadfe0..6a0006ef04c8351c806b17c5100c094281efa946 100644 (file)
@@ -1386,7 +1386,7 @@ static void gpencil_primitive_edit_event_handling(
     case LEFTMOUSE: {
       if ((event->val == KM_RELEASE) && (tgpi->flag == IN_PROGRESS)) {
         /* set control points and enter edit mode */
-        if ((ELEM(tgpi->type, GP_STROKE_POLYLINE))) {
+        if (ELEM(tgpi->type, GP_STROKE_POLYLINE)) {
           gpencil_primitive_add_segment(tgpi);
           copy_v2_v2(tgpi->start, tgpi->end);
           copy_v2_v2(tgpi->origin, tgpi->start);
index bfe413ccb9111e1db3e3ef642030508534fe6d58..73fd45693a2de3dd33b7afaf7960aad07157fa34 100644 (file)
@@ -309,7 +309,7 @@ static void quadriflow_update_job(void *customdata, float progress, int *cancel)
 
 static Mesh *remesh_symmetry_bisect(Main *bmain, Mesh *mesh, eSymmetryAxes symmetry_axes)
 {
-  MirrorModifierData mmd = {0};
+  MirrorModifierData mmd = {{0}};
   mmd.tolerance = QUADRIFLOW_MIRROR_BISECT_TOLERANCE;
 
   Mesh *mesh_bisect, *mesh_bisect_temp;
@@ -343,7 +343,7 @@ static Mesh *remesh_symmetry_bisect(Main *bmain, Mesh *mesh, eSymmetryAxes symme
 
 static Mesh *remesh_symmetry_mirror(Object *ob, Mesh *mesh, eSymmetryAxes symmetry_axes)
 {
-  MirrorModifierData mmd = {0};
+  MirrorModifierData mmd = {{0}};
   mmd.tolerance = QUADRIFLOW_MIRROR_BISECT_TOLERANCE;
   Mesh *mesh_mirror, *mesh_mirror_temp;
 
index 1a3f57a1ed7f1f5fb9cc25d9677adabca09e3a00..5f3992852f04c6688985aa8a2c819017b6beb04e 100644 (file)
@@ -8379,7 +8379,7 @@ static int sculpt_symmetrize_exec(bContext *C, wmOperator *UNUSED(op))
       ED_sculpt_undo_geometry_begin(ob, "mesh symmetrize");
       Mesh *mesh = ob->data;
       Mesh *mesh_mirror;
-      MirrorModifierData mmd = {0};
+      MirrorModifierData mmd = {{0}};
       int axis = 0;
       mmd.flag = 0;
       mmd.tolerance = 0.005f;
index 73124768696e4feb00300679aa6fa7eab4e2e341..a8ed2c5f2a587e6778d792a4824ebcd7d94ffcee 100644 (file)
@@ -449,8 +449,6 @@ void AbstractHierarchyIterator::make_writers(const HierarchyContext *parent_cont
     unit_m4(parent_matrix_inv_world);
   }
 
-  const std::string &parent_export_path = parent_context ? parent_context->export_path : "";
-
   for (HierarchyContext *context : graph_children(parent_context)) {
     copy_m4_m4(context->parent_matrix_inv_world, parent_matrix_inv_world);
 
index 0c2ce9783eca4fb73642d177678af23c9c7f52a9..97ca879736ecf4f61205c1446b0438e4519c4f62 100644 (file)
@@ -339,14 +339,14 @@ void WM_msg_subscribe_ID(struct wmMsgBus *mbus,
                          const wmMsgSubscribeValue *msg_val_params,
                          const char *id_repr)
 {
-  wmMsgParams_RNA msg_key_params = {NULL};
+  wmMsgParams_RNA msg_key_params = {{NULL}};
   RNA_id_pointer_create(id, &msg_key_params.ptr);
   WM_msg_subscribe_rna_params(mbus, &msg_key_params, msg_val_params, id_repr);
 }
 
 void WM_msg_publish_ID(struct wmMsgBus *mbus, ID *id)
 {
-  wmMsgParams_RNA msg_key_params = {NULL};
+  wmMsgParams_RNA msg_key_params = {{NULL}};
   RNA_id_pointer_create(id, &msg_key_params.ptr);
   WM_msg_publish_rna_params(mbus, &msg_key_params);
 }