[Bf-blender-cvs] [3ead74716ac] soc-2016-pbvh-painting: Cleanup: remove redundant headers-in-headers

Campbell Barton noreply at git.blender.org
Mon Apr 10 05:33:54 CEST 2017


Commit: 3ead74716ac69e23031cc294b3932e246f2b203a
Author: Campbell Barton
Date:   Mon Apr 10 13:33:25 2017 +1000
Branches: soc-2016-pbvh-painting
https://developer.blender.org/rB3ead74716ac69e23031cc294b3932e246f2b203a

Cleanup: remove redundant headers-in-headers

This brings the branch closer to master.

Also remove unused 'indexar' & 'accumulate_stack'.

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

M	source/blender/editors/sculpt_paint/paint_intern.h
M	source/blender/editors/sculpt_paint/paint_vertex.c
M	source/blender/editors/sculpt_paint/sculpt.c
M	source/blender/editors/sculpt_paint/sculpt_intern.h
M	source/blender/editors/sculpt_paint/sculpt_undo.c
M	source/blender/makesrna/intern/rna_sculpt_paint.c

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

diff --git a/source/blender/editors/sculpt_paint/paint_intern.h b/source/blender/editors/sculpt_paint/paint_intern.h
index dcbe7bc5961..56f7b3db48e 100644
--- a/source/blender/editors/sculpt_paint/paint_intern.h
+++ b/source/blender/editors/sculpt_paint/paint_intern.h
@@ -54,7 +54,6 @@ struct wmOperator;
 struct wmOperatorType;
 struct wmWindowManager;
 struct DMCoNo;
-struct BLI_Stack;
 struct MeshElemMap;
 enum PaintMode;
 
@@ -144,7 +143,6 @@ typedef struct WeightPaintInfo {
 typedef struct VPaintData {
 	struct ViewContext *vc;
 	unsigned int paintcol;
-	int *indexar;
 
 	struct VertProjHandle *vp_handle;
 	struct DMCoNo *vertexcosnos;
@@ -164,7 +162,6 @@ typedef struct VPaintData {
 
 typedef struct WPaintData {
 	struct ViewContext *vc;
-	int *indexar;
 
 	struct WeightPaintGroupData active, mirror;
 
@@ -188,8 +185,6 @@ typedef struct WPaintData {
 		int *vmap_mem;
 	} blur_data;
 
-	struct BLI_Stack *accumulate_stack;  /* for reuse (WPaintDefer) */
-
 	int defbase_tot;
 } WPaintData;
 
diff --git a/source/blender/editors/sculpt_paint/paint_vertex.c b/source/blender/editors/sculpt_paint/paint_vertex.c
index c3eb5288276..b80b745496a 100644
--- a/source/blender/editors/sculpt_paint/paint_vertex.c
+++ b/source/blender/editors/sculpt_paint/paint_vertex.c
@@ -35,6 +35,7 @@
 #include "BLI_math.h"
 #include "BLI_array_utils.h"
 #include "BLI_bitmap.h"
+#include "BLI_task.h"
 #include "BLI_string_utils.h"
 
 #include "IMB_imbuf.h"
diff --git a/source/blender/editors/sculpt_paint/sculpt.c b/source/blender/editors/sculpt_paint/sculpt.c
index 08443d89ab5..746c560090d 100644
--- a/source/blender/editors/sculpt_paint/sculpt.c
+++ b/source/blender/editors/sculpt_paint/sculpt.c
@@ -37,6 +37,8 @@
 
 #include "BLI_math.h"
 #include "BLI_blenlib.h"
+#include "BLI_dial.h"
+#include "BLI_task.h"
 #include "BLI_utildefines.h"
 #include "BLI_ghash.h"
 
@@ -47,6 +49,8 @@
 #include "DNA_meshdata_types.h"
 #include "DNA_node_types.h"
 #include "DNA_object_types.h"
+#include "DNA_scene_types.h"
+#include "DNA_brush_types.h"
 
 #include "BKE_pbvh.h"
 #include "BKE_brush.h"
@@ -74,7 +78,7 @@
 #include "ED_sculpt.h"
 #include "ED_object.h"
 #include "ED_screen.h"
-
+#include "ED_view3d.h"
 #include "paint_intern.h"
 #include "sculpt_intern.h"
 
@@ -87,6 +91,9 @@
 #include "UI_interface.h"
 #include "UI_resources.h"
 
+#include "bmesh.h"
+#include "bmesh_tools.h"
+
 #include <math.h>
 #include <stdlib.h>
 #include <string.h>
diff --git a/source/blender/editors/sculpt_paint/sculpt_intern.h b/source/blender/editors/sculpt_paint/sculpt_intern.h
index 6b5c1bf7b95..b72404f974f 100644
--- a/source/blender/editors/sculpt_paint/sculpt_intern.h
+++ b/source/blender/editors/sculpt_paint/sculpt_intern.h
@@ -36,26 +36,12 @@
 #include "DNA_listBase.h"
 #include "DNA_vec_types.h"
 #include "DNA_key_types.h"
-#include "DNA_brush_types.h"
-#include "DNA_view3d_types.h"
-#include "DNA_screen_types.h"
-#include "DNA_scene_types.h"
-#include "DNA_mesh_types.h"
 
 #include "BLI_bitmap.h"
-#include "BLI_dial.h"
-#include "BLI_task.h"
 #include "BLI_threads.h"
 
 #include "BKE_pbvh.h"
 
-#include "ED_view3d.h"
-
-#include "paint_intern.h"
-
-#include "bmesh.h"
-#include "bmesh_tools.h"
-
 struct bContext;
 struct KeyBlock;
 struct Object;
@@ -134,7 +120,7 @@ typedef struct SculptUndoNode {
 /************** Access to original unmodified vertex data *************/
 
 typedef struct SculptOrigVertData {
-	BMLog *bm_log;
+	struct BMLog *bm_log;
 
 	SculptUndoNode *unode;
 	float(*coords)[3];
@@ -183,19 +169,19 @@ typedef struct SculptProjectVector {
 /* Single struct used by all BLI_task threaded callbacks, let's avoid adding 10's of those... */
 typedef struct SculptThreadedTaskData {
 	bContext *C;
-	Sculpt *sd;
-	Object *ob;
-	Brush *brush;
-	PBVHNode **nodes;
+	struct Sculpt *sd;
+	struct Object *ob;
+	struct Brush *brush;
+	struct PBVHNode **nodes;
 	int totnode;
 
-	VPaint *vp;
-	VPaintData *vpd;
-	WPaintData *wpd;
-	WeightPaintInfo *wpi;
-	unsigned int* lcol;
+	struct VPaint *vp;
+	struct VPaintData *vpd;
+	struct WPaintData *wpd;
+	struct WeightPaintInfo *wpi;
+	unsigned int *lcol;
 	struct MeshElemMap **vertToLoopMaps;
-	Mesh *me;
+	struct Mesh *me;
 
 
 	/* Data specific to some callbacks. */
@@ -234,12 +220,12 @@ typedef struct SculptBrushTest {
 	int mirror_symmetry_pass;
 
 	/* View3d clipping - only set rv3d for clipping */
-	RegionView3D *clip_rv3d;
+	struct RegionView3D *clip_rv3d;
 } SculptBrushTest;
 
 typedef struct {
-	Sculpt *sd;
-	SculptSession *ss;
+	struct Sculpt *sd;
+	struct SculptSession *ss;
 	float radius_squared;
 	bool original;
 } SculptSearchSphereData;
@@ -250,13 +236,14 @@ bool sculpt_brush_test_sq(SculptBrushTest *test, const float co[3]);
 bool sculpt_brush_test_fast(const SculptBrushTest *test, const float co[3]);
 bool sculpt_brush_test_cube(SculptBrushTest *test, const float co[3], float local[4][4]);
 bool sculpt_search_sphere_cb(PBVHNode *node, void *data_v);
-float tex_strength(SculptSession *ss, Brush *br,
-	const float point[3],
-	const float len,
-	const short vno[3],
-	const float fno[3],
-	const float mask,
-	const int thread_id);
+float tex_strength(
+        SculptSession *ss, struct Brush *br,
+        const float point[3],
+        const float len,
+        const short vno[3],
+        const float fno[3],
+        const float mask,
+        const int thread_id);
 
 
 /* Cache stroke properties. Used because
@@ -297,8 +284,8 @@ typedef struct StrokeCache {
 	float projection_mat[4][4];
 
 	/* Clean this up! */
-	ViewContext *vc;
-	Brush *brush;
+	struct ViewContext *vc;
+	struct Brush *brush;
 
 	float special_rotation;
 	float grab_delta[3], grab_delta_symmetry[3];
@@ -309,8 +296,9 @@ typedef struct StrokeCache {
 	bool is_rake_rotation_valid;
 	struct SculptRakeData rake_data;
 
-	int symmetry; /* Symmetry index between 0 and 7 bit combo 0 is Brush only;
-				  * 1 is X mirror; 2 is Y mirror; 3 is XY; 4 is Z; 5 is XZ; 6 is YZ; 7 is XYZ */
+	/* Symmetry index between 0 and 7 bit combo 0 is Brush only;
+	 * 1 is X mirror; 2 is Y mirror; 3 is XY; 4 is Z; 5 is XZ; 6 is YZ; 7 is XYZ */
+	int symmetry;
 	int mirror_symmetry_pass; /* the symmetry pass we are currently on between 0 and 7*/
 	float true_view_normal[3];
 	float view_normal[3];
@@ -336,7 +324,7 @@ typedef struct StrokeCache {
 	float anchored_location[3];
 
 	float vertex_rotation; /* amount to rotate the vertices when using rotate brush */
-	Dial *dial;
+	struct Dial *dial;
 
 	char saved_active_brush_name[MAX_ID_NAME];
 	char saved_mask_brush_tool;
@@ -365,7 +353,7 @@ void sculpt_vertcos_to_key(Object *ob, KeyBlock *kb, float (*vertCos)[3]);
 
 void sculpt_update_object_bounding_box(struct Object *ob);
 
-bool sculpt_get_redraw_rect(ARegion *ar, RegionView3D *rv3d, Object *ob, rcti *rect);
+bool sculpt_get_redraw_rect(struct ARegion *ar, struct RegionView3D *rv3d, Object *ob, rcti *rect);
 
 #define SCULPT_THREADED_LIMIT 4
 
diff --git a/source/blender/editors/sculpt_paint/sculpt_undo.c b/source/blender/editors/sculpt_paint/sculpt_undo.c
index 2a8af934c6e..44bd872d107 100644
--- a/source/blender/editors/sculpt_paint/sculpt_undo.c
+++ b/source/blender/editors/sculpt_paint/sculpt_undo.c
@@ -46,6 +46,7 @@
 
 #include "DNA_meshdata_types.h"
 #include "DNA_object_types.h"
+#include "DNA_scene_types.h"
 #include "DNA_mesh_types.h"
 
 #include "BKE_ccg.h"
diff --git a/source/blender/makesrna/intern/rna_sculpt_paint.c b/source/blender/makesrna/intern/rna_sculpt_paint.c
index 970eac0db04..98dbfa3ae5f 100644
--- a/source/blender/makesrna/intern/rna_sculpt_paint.c
+++ b/source/blender/makesrna/intern/rna_sculpt_paint.c
@@ -692,7 +692,7 @@ static void rna_def_vertex_paint(BlenderRNA *brna)
 	RNA_def_property_range(prop, 1, 64);
 	RNA_def_property_ui_range(prop, 1, 32, 1, 1);
 	RNA_def_property_ui_text(prop, "Radial Symmetry Count X Axis",
-    "Number of times to copy strokes across the surface");
+	                         "Number of times to copy strokes across the surface");
 }
 
 static void rna_def_image_paint(BlenderRNA *brna)




More information about the Bf-blender-cvs mailing list