[Bf-blender-cvs] SVN commit: /data/svn/bf-blender [30539] trunk/blender: * can use file for brush icon

Jason Wilkins Jason.A.Wilkins at gmail.com
Tue Jul 20 15:42:27 CEST 2010


Revision: 30539
          http://projects.blender.org/plugins/scmsvn/viewcvs.php?view=rev&root=bf-blender&revision=30539
Author:   jwilkins
Date:     2010-07-20 15:42:27 +0200 (Tue, 20 Jul 2010)

Log Message:
-----------
* can use file for brush icon
* fixed memory leaks
* moved some of the brush icon code around
* the update of the icon after a change is more responsive

Modified Paths:
--------------
    trunk/blender/release/scripts/ui/space_view3d_toolbar.py
    trunk/blender/source/blender/blenkernel/intern/brush.c
    trunk/blender/source/blender/blenloader/intern/readfile.c
    trunk/blender/source/blender/editors/render/render_preview.c
    trunk/blender/source/blender/makesrna/intern/rna_brush.c

Modified: trunk/blender/release/scripts/ui/space_view3d_toolbar.py
===================================================================
--- trunk/blender/release/scripts/ui/space_view3d_toolbar.py	2010-07-20 13:32:42 UTC (rev 30538)
+++ trunk/blender/release/scripts/ui/space_view3d_toolbar.py	2010-07-20 13:42:27 UTC (rev 30539)
@@ -1126,8 +1126,8 @@
         row = col.row(align=True)
         row.prop(brush, "icon", text="")
 
-        #row = col.row(align=True)
-        #row.prop(brush, "icon_filepath", text="")
+        row = col.row(align=True)
+        row.prop(brush, "icon_filepath", text="")
 
 # ********** default tools for weightpaint ****************
 

Modified: trunk/blender/source/blender/blenkernel/intern/brush.c
===================================================================
--- trunk/blender/source/blender/blenkernel/intern/brush.c	2010-07-20 13:32:42 UTC (rev 30538)
+++ trunk/blender/source/blender/blenkernel/intern/brush.c	2010-07-20 13:42:27 UTC (rev 30539)
@@ -104,8 +104,6 @@
 
 	/* brush appearance  */
 
-	brush->icon_imbuf= get_brush_icon(brush);
-
 	brush->add_col[0]= 1.00; /* add mode color is light red */
 	brush->add_col[1]= 0.39;
 	brush->add_col[2]= 0.39;
@@ -130,9 +128,11 @@
 	
 	brushn= copy_libblock(brush);
 
-	if(brush->mtex.tex) id_us_plus((ID*)brush->mtex.tex);
+	if (brush->mtex.tex)
+		id_us_plus((ID*)brush->mtex.tex);
 
-	IMB_refImBuf(brushn->icon_imbuf);
+	if (brush->icon_imbuf)
+		brushn->icon_imbuf= IMB_dupImBuf(brush->icon_imbuf);
 
 	brushn->curve= curvemapping_copy(brush->curve);
 
@@ -151,7 +151,7 @@
 	if (brush->mtex.tex)
 		brush->mtex.tex->id.us--;
 
-	if (brush->icon==BRUSH_ICON_FILE && brush->icon_imbuf)
+	if (brush->icon_imbuf)
 		IMB_freeImBuf(brush->icon_imbuf);
 
 	BKE_previewimg_free(&(brush->preview));
@@ -211,140 +211,6 @@
 	}
 }
 
-/* image icon function */
-ImBuf* get_brush_icon(Brush *brush)
-{
-	extern char datatoc_blob_png;
-	extern char datatoc_clay_png;
-	extern char datatoc_crease_png;
-	extern char datatoc_draw_png;
-	extern char datatoc_fill_png;
-	extern char datatoc_flatten_png;
-	extern char datatoc_grab_png;
-	extern char datatoc_inflate_png;
-	extern char datatoc_layer_png;
-	extern char datatoc_nudge_png;
-	extern char datatoc_pinch_png;
-	extern char datatoc_scrape_png;
-	extern char datatoc_smooth_png;
-	extern char datatoc_snake_hook_png;
-	extern char datatoc_thumb_png;
-	extern char datatoc_twist_png;
-
-	extern int datatoc_blob_png_size;
-	extern int datatoc_clay_png_size;
-	extern int datatoc_crease_png_size;
-	extern int datatoc_draw_png_size;
-	extern int datatoc_fill_png_size;
-	extern int datatoc_flatten_png_size;
-	extern int datatoc_grab_png_size;
-	extern int datatoc_inflate_png_size;
-	extern int datatoc_layer_png_size;
-	extern int datatoc_nudge_png_size;
-	extern int datatoc_pinch_png_size;
-	extern int datatoc_scrape_png_size;
-	extern int datatoc_smooth_png_size;
-	extern int datatoc_snake_hook_png_size;
-	extern int datatoc_thumb_png_size;
-	extern int datatoc_twist_png_size;
-
-	void *icon_data[]= {
-		0,
-		&datatoc_blob_png,
-		&datatoc_clay_png,
-		&datatoc_crease_png,
-		&datatoc_draw_png,
-		&datatoc_fill_png,
-		&datatoc_flatten_png,
-		&datatoc_grab_png,
-		&datatoc_inflate_png,
-		&datatoc_layer_png,
-		&datatoc_nudge_png,
-		&datatoc_pinch_png,
-		&datatoc_scrape_png,
-		&datatoc_smooth_png,
-		&datatoc_snake_hook_png,
-		&datatoc_thumb_png,
-		&datatoc_twist_png,
-	};
-
-	size_t icon_size[]= {
-		0,
-		datatoc_blob_png_size,
-		datatoc_clay_png_size,
-		datatoc_crease_png_size,
-		datatoc_draw_png_size,
-		datatoc_fill_png_size,
-		datatoc_flatten_png_size,
-		datatoc_grab_png_size,
-		datatoc_inflate_png_size,
-		datatoc_layer_png_size,
-		datatoc_nudge_png_size,
-		datatoc_pinch_png_size,
-		datatoc_scrape_png_size,
-		datatoc_smooth_png_size,
-		datatoc_snake_hook_png_size,
-		datatoc_thumb_png_size,
-		datatoc_twist_png_size,
-	};
-
-	static ImBuf  *icon_imbuf[BRUSH_ICON_COUNT]= { 0 };
-
-	static const int flags = IB_rect|IB_multilayer|IB_metadata;
-
-	static const int default_icon = BRUSH_ICON_DRAW;
-
-	char path[240];
-	char *folder;
-
-	if (!(brush->icon_imbuf)) {
-		if (brush->icon==BRUSH_ICON_FILE) {
-
-			if (brush->icon_filepath[0]) {
-				// first use the path directly to try and load the file
-
-				BLI_strncpy(path, brush->icon_filepath, sizeof(brush->icon_filepath));
-				BLI_path_abs(path, G.sce);
-
-				brush->icon_imbuf= IMB_loadiffname(path, flags);
-
-				// otherwise lets try to find it in other directories
-				if (!(brush->icon_imbuf)) {
-					folder= BLI_get_folder(BLENDER_DATAFILES, "brushicons");
-
-					path[0]= 0;
-
-					BLI_make_file_string(G.sce, path, folder, brush->icon_filepath);
-
-					if (path[0])
-						brush->icon_imbuf= IMB_loadiffname(path, flags);
-				}
-			}
-
-			// if all else fails use a default image
-			if (!(brush->icon_imbuf)) {
-				if (!icon_imbuf[default_icon])
-					icon_imbuf[default_icon]= IMB_ibImageFromMemory(icon_data[default_icon], icon_size[default_icon], flags);
-
-				brush->icon_imbuf= icon_imbuf[default_icon];
-			}
-		}
-		else {
-			if (!icon_imbuf[brush->icon])
-				icon_imbuf[brush->icon]= IMB_ibImageFromMemory(icon_data[brush->icon], icon_size[brush->icon], flags);
-
-			brush->icon_imbuf= icon_imbuf[brush->icon];
-		}
-
-		BKE_icon_changed(BKE_icon_getid(&(brush->id)));
-	}
-
-	if (!(brush->icon_imbuf))
-		printf("get_brush_icon: unable to resolve brush icon imbuf\n");
-
-	return brush->icon_imbuf;
-}
-
 /* Library Operations */
 
 int brush_set_nr(Brush **current_brush, int nr, const char *name)

Modified: trunk/blender/source/blender/blenloader/intern/readfile.c
===================================================================
--- trunk/blender/source/blender/blenloader/intern/readfile.c	2010-07-20 13:32:42 UTC (rev 30538)
+++ trunk/blender/source/blender/blenloader/intern/readfile.c	2010-07-20 13:42:27 UTC (rev 30539)
@@ -1540,9 +1540,6 @@
 
 			brush->mtex.tex= newlibadr_us(fd, brush->id.lib, brush->mtex.tex);
 			brush->clone.image= newlibadr_us(fd, brush->id.lib, brush->clone.image);
-
-			brush->preview=0;
-			brush->icon_imbuf=0;
 		}
 	}
 }
@@ -1557,6 +1554,9 @@
 		direct_link_curvemapping(fd, brush->curve);
 	else
 		brush_curve_preset(brush, CURVE_PRESET_SHARP);
+
+	brush->preview= NULL;
+	brush->icon_imbuf= NULL;
 }
 
 static void direct_link_script(FileData *fd, Script *script)

Modified: trunk/blender/source/blender/editors/render/render_preview.c
===================================================================
--- trunk/blender/source/blender/editors/render/render_preview.c	2010-07-20 13:32:42 UTC (rev 30538)
+++ trunk/blender/source/blender/editors/render/render_preview.c	2010-07-20 13:42:27 UTC (rev 30539)
@@ -101,6 +101,131 @@
 static int qtest() {return 0;}
 /* XXX */
 
+ImBuf* get_brush_icon(Brush *brush)
+{
+	extern char datatoc_blob_png;
+	extern char datatoc_clay_png;
+	extern char datatoc_crease_png;
+	extern char datatoc_draw_png;
+	extern char datatoc_fill_png;
+	extern char datatoc_flatten_png;
+	extern char datatoc_grab_png;
+	extern char datatoc_inflate_png;
+	extern char datatoc_layer_png;
+	extern char datatoc_nudge_png;
+	extern char datatoc_pinch_png;
+	extern char datatoc_scrape_png;
+	extern char datatoc_smooth_png;
+	extern char datatoc_snake_hook_png;
+	extern char datatoc_thumb_png;
+	extern char datatoc_twist_png;
+
+	extern int datatoc_blob_png_size;
+	extern int datatoc_clay_png_size;
+	extern int datatoc_crease_png_size;
+	extern int datatoc_draw_png_size;
+	extern int datatoc_fill_png_size;
+	extern int datatoc_flatten_png_size;
+	extern int datatoc_grab_png_size;
+	extern int datatoc_inflate_png_size;
+	extern int datatoc_layer_png_size;
+	extern int datatoc_nudge_png_size;
+	extern int datatoc_pinch_png_size;
+	extern int datatoc_scrape_png_size;
+	extern int datatoc_smooth_png_size;
+	extern int datatoc_snake_hook_png_size;
+	extern int datatoc_thumb_png_size;
+	extern int datatoc_twist_png_size;
+
+	void *icon_data[]= {
+		0,
+		&datatoc_blob_png,
+		&datatoc_clay_png,
+		&datatoc_crease_png,
+		&datatoc_draw_png,
+		&datatoc_fill_png,
+		&datatoc_flatten_png,
+		&datatoc_grab_png,
+		&datatoc_inflate_png,
+		&datatoc_layer_png,
+		&datatoc_nudge_png,
+		&datatoc_pinch_png,
+		&datatoc_scrape_png,
+		&datatoc_smooth_png,
+		&datatoc_snake_hook_png,
+		&datatoc_thumb_png,
+		&datatoc_twist_png,
+	};
+
+	size_t icon_size[]= {
+		0,
+		datatoc_blob_png_size,
+		datatoc_clay_png_size,
+		datatoc_crease_png_size,
+		datatoc_draw_png_size,
+		datatoc_fill_png_size,
+		datatoc_flatten_png_size,
+		datatoc_grab_png_size,
+		datatoc_inflate_png_size,
+		datatoc_layer_png_size,
+		datatoc_nudge_png_size,
+		datatoc_pinch_png_size,
+		datatoc_scrape_png_size,
+		datatoc_smooth_png_size,
+		datatoc_snake_hook_png_size,
+		datatoc_thumb_png_size,
+		datatoc_twist_png_size,
+	};
+
+	static const int flags = IB_rect|IB_multilayer|IB_metadata;
+
+	static const int default_icon = BRUSH_ICON_DRAW;
+
+	char path[240];
+	char *folder;
+
+	if (!(brush->icon_imbuf)) {
+		if (brush->icon==BRUSH_ICON_FILE) {
+
+			if (brush->icon_filepath[0]) {
+				// first use the path directly to try and load the file
+
+				BLI_strncpy(path, brush->icon_filepath, sizeof(brush->icon_filepath));
+				BLI_path_abs(path, G.sce);
+
+				brush->icon_imbuf= IMB_loadiffname(path, flags);
+
+				// otherwise lets try to find it in other directories
+				if (!(brush->icon_imbuf)) {
+					folder= BLI_get_folder(BLENDER_DATAFILES, "brushicons");
+
+					path[0]= 0;
+
+					BLI_make_file_string(G.sce, path, folder, brush->icon_filepath);
+
+					if (path[0])
+						brush->icon_imbuf= IMB_loadiffname(path, flags);
+				}
+			}
+
+			// if all else fails use a default image
+			if (!(brush->icon_imbuf))
+				brush->icon_imbuf= IMB_ibImageFromMemory(icon_data[default_icon], icon_size[default_icon], flags);
+		}
+		else {
+			if (!(brush->icon_imbuf))
+				brush->icon_imbuf= IMB_ibImageFromMemory(icon_data[brush->icon], icon_size[brush->icon], flags);
+		}
+
+		BKE_icon_changed(BKE_icon_getid(&(brush->id)));
+	}
+
+	if (!(brush->icon_imbuf))
+		printf("get_brush_icon: unable to resolve brush icon imbuf\n");
+
+	return brush->icon_imbuf;
+}
+
 typedef struct ShaderPreview {
 	/* from wmJob */
 	void *owner;
@@ -1114,6 +1239,14 @@
 		shader_preview_startjob(customdata, stop, do_update);
 }
 
+static void common_preview_endjob(void *customdata)
+{
+	ShaderPreview *sp= customdata;
+
+	if(sp->id && GS(sp->id->name) == ID_BR)
+		WM_main_add_notifier(NC_BRUSH|NA_EDITED, sp->id);
+}
+
 /* exported functions */
 

@@ Diff output truncated at 10240 characters. @@




More information about the Bf-blender-cvs mailing list