[Bf-blender-cvs] [5af58fa] master: Fix compilation error caused by wrong array initialization

Sergey Sharybin noreply at git.blender.org
Mon Aug 29 09:41:56 CEST 2016


Commit: 5af58faf799e3fd19ca9319f7ebef0654967bdc0
Author: Sergey Sharybin
Date:   Mon Aug 29 09:40:15 2016 +0200
Branches: master
https://developer.blender.org/rB5af58faf799e3fd19ca9319f7ebef0654967bdc0

Fix compilation error caused by wrong array initialization

We usually don't silence migh-be-uninitialized warning (which is the only
thing which could explain setting matrix to all zeroes) so we can catch
such errors when using tools like Valgrind.

I don't get warning here and the initializer was wrong, so removing it.

If it-s _REALLY_ needed please do a proper initialization.

===================================================================

M	source/blender/editors/gpencil/gpencil_edit.c

===================================================================

diff --git a/source/blender/editors/gpencil/gpencil_edit.c b/source/blender/editors/gpencil/gpencil_edit.c
index 99f4ca2..9f700e8 100644
--- a/source/blender/editors/gpencil/gpencil_edit.c
+++ b/source/blender/editors/gpencil/gpencil_edit.c
@@ -1894,7 +1894,7 @@ static int gp_strokes_reproject_poll(bContext *C)
 	return (gp_stroke_edit_poll(C) && ED_operator_view3d_active(C));
 }
 
-static int gp_strokes_reproject_exec(bContext *C, wmOperator *op)
+static int gp_strokes_reproject_exec(bContext *C, wmOperator *UNUSED(op))
 {
 	Scene *scene = CTX_data_scene(C);
 	GP_SpaceConversion gsc = {NULL};
@@ -1908,7 +1908,7 @@ static int gp_strokes_reproject_exec(bContext *C, wmOperator *op)
 		if (gps->flag & GP_STROKE_SELECT) {
 			bGPDspoint *pt;
 			int i;
-			float inverse_diff_mat[4][4] = {0.0f};
+			float inverse_diff_mat[4][4];
 			
 			/* Compute inverse matrix for unapplying parenting once instead of doing per-point */
 			/* TODO: add this bit to the iteration macro? */




More information about the Bf-blender-cvs mailing list