From 8354267448dab01626da0c4b3d2eedfa20e6858a Mon Sep 17 00:00:00 2001
From: Thierry Lorthiois <lorthiois@bbsoft.fr>
Date: Sat, 19 Sep 2009 20:56:15 +0000
Subject: [PATCH] lower cpu use with icon. replace HUE by ALPHA on icon (see
 task_icon_asb).

---
 src/config.c       |   8 +--
 src/taskbar/task.c | 167 +++++++++++++++++----------------------------
 src/taskbar/task.h |   9 ++-
 src/tint.c         |   4 +-
 src/util/common.c  |  10 +--
 src/util/common.h  |   6 +-
 6 files changed, 82 insertions(+), 122 deletions(-)

diff --git a/src/config.c b/src/config.c
index 9fda951..da69dea 100644
--- a/src/config.c
+++ b/src/config.c
@@ -530,15 +530,15 @@ void add_entry (char *key, char *value)
 		if (value2) panel_config->g_task.font_active.alpha = (atoi (value2) / 100.0);
 		else panel_config->g_task.font_active.alpha = 0.1;
 	}
-	else if (strcmp (key, "task_icon_hsb") == 0) {
+	else if (strcmp (key, "task_icon_asb") == 0) {
 		extract_values(value, &value1, &value2, &value3);
-		panel_config->g_task.hue = atoi(value1);
+		panel_config->g_task.alpha = atoi(value1);
 		panel_config->g_task.saturation = atoi(value2);
 		panel_config->g_task.brightness = atoi(value3);
 	}
-	else if (strcmp (key, "task_active_icon_hsb") == 0) {
+	else if (strcmp (key, "task_active_icon_asb") == 0) {
 		extract_values(value, &value1, &value2, &value3);
-		panel_config->g_task.hue_active = atoi(value1);
+		panel_config->g_task.alpha_active = atoi(value1);
 		panel_config->g_task.saturation_active = atoi(value2);
 		panel_config->g_task.brightness_active = atoi(value3);
 	}
diff --git a/src/taskbar/task.c b/src/taskbar/task.c
index 2df02fc..500d7f5 100644
--- a/src/taskbar/task.c
+++ b/src/taskbar/task.c
@@ -51,8 +51,7 @@ Task *add_task (Window win)
 	// allocate only one title and one icon
 	// even with task_on_all_desktop and with task_on_all_panel
 	new_tsk.title = 0;
-	new_tsk.icon_data = 0;
-	new_tsk.icon_data_active = 0;
+	new_tsk.icon = new_tsk.icon_active = NULL;
 	get_title(&new_tsk);
 	get_icon(&new_tsk);
 
@@ -74,8 +73,8 @@ Task *add_task (Window win)
 			new_tsk2->win = new_tsk.win;
 			new_tsk2->desktop = new_tsk.desktop;
 			new_tsk2->title = new_tsk.title;
-			new_tsk2->icon_data = new_tsk.icon_data;
-			new_tsk2->icon_data_active = new_tsk.icon_data_active;
+			new_tsk2->icon = new_tsk.icon;
+			new_tsk2->icon_active = new_tsk.icon_active;
 			new_tsk2->icon_width = new_tsk.icon_width;
 			new_tsk2->icon_height = new_tsk.icon_height;
 			tskbar->area.list = g_slist_append(tskbar->area.list, new_tsk2);
@@ -99,9 +98,12 @@ void remove_task (Task *tsk)
 	//printf("remove_task %s %d\n", tsk->title, tsk->desktop);
 	if (tsk->title)
 		free (tsk->title);
-	if (tsk->icon_data) {
-		free (tsk->icon_data);
-		free (tsk->icon_data_active);
+	if (tsk->icon) {
+		imlib_context_set_image(tsk->icon);
+		imlib_free_image();
+		imlib_context_set_image(tsk->icon_active);
+		imlib_free_image();
+		tsk->icon = tsk->icon_active = NULL;
 	}
 
 	int i, j;
@@ -172,43 +174,42 @@ void get_icon (Task *tsk)
 {
 	Panel *panel = tsk->area.panel;
 	if (!panel->g_task.icon) return;
-	int num;
-	long *data;
 
-	if (tsk->icon_data) {
-		free (tsk->icon_data);
-		free (tsk->icon_data_active);
-		tsk->icon_data = tsk->icon_data_active = 0;
+	if (tsk->icon) {
+		imlib_context_set_image(tsk->icon);
+		imlib_free_image();
+		imlib_context_set_image(tsk->icon_active);
+		imlib_free_image();
+		tsk->icon = tsk->icon_active = NULL;
 	}
 	tsk->area.redraw = 1;
 
-	data = server_get_property (tsk->win, server.atom._NET_WM_ICON, XA_CARDINAL, &num);
+	int i;
+	Imlib_Image img = NULL;
+	long *data = server_get_property (tsk->win, server.atom._NET_WM_ICON, XA_CARDINAL, &i);
 	if (data) {
 		// get ARGB icon
 		int w, h;
 		long *tmp_data;
-		tmp_data = get_best_icon (data, get_icon_count (data, num), num, &w, &h, panel->g_task.icon_size1);
 
-		tsk->icon_width = w;
-		tsk->icon_height = h;
-		// DATA32 is provided by imlib2
-		tsk->icon_data = malloc (w * h * sizeof (DATA32));
+		tmp_data = get_best_icon (data, get_icon_count (data, i), i, &w, &h, panel->g_task.icon_size1);
 
-		if (tsk->icon_data) {
 #ifdef __x86_64__
-			int i, length = w * h;
-			for (i = 0; i < length; ++i)
-				tsk->icon_data[i] =  tmp_data[i];
+		DATA32 *icon_data = malloc (w * h * sizeof (DATA32));
+		int length = w * h;
+		for (i = 0; i < length; ++i)
+			icon_data[i] =  tmp_data[i];
+		img = imlib_create_image_using_data (w, h, icon_data);
 #else
-			memcpy (tsk->icon_data, tmp_data, w * h * sizeof (DATA32));
+		img = imlib_create_image_using_data (w, h, (DATA32*)tmp_data);
 #endif
-		}
+		imlib_context_set_image(img);
+		imlib_image_set_has_alpha(1);
 		XFree (data);
 	}
 	else {
 		// get Pixmap icon
 		XWMHints *hints = XGetWMHints(server.dsp, tsk->win);
-		Imlib_Image  img = NULL;
 		if (hints) {
 			if (hints->flags & IconPixmapHint && hints->icon_pixmap != 0) {
 				// get width, height and depth for the pixmap
@@ -217,75 +218,48 @@ void get_icon (Task *tsk)
 				uint border_width, bpp;
 				uint w, h;
 
-				// printf("  get pixmap\n");
+				//printf("  get pixmap\n");
 				XGetGeometry(server.dsp, hints->icon_pixmap, &root, &icon_x, &icon_y, &w, &h, &border_width, &bpp);
 				imlib_context_set_drawable(hints->icon_pixmap);
 				img = imlib_create_image_from_drawable(hints->icon_mask, 0, 0, w, h, 0);
-				imlib_context_set_image(img);
 			}
-			else
-				imlib_context_set_image(default_icon);
 		}
-		else
-			imlib_context_set_image(default_icon);
-		data = imlib_image_get_data();
-		tsk->icon_width = imlib_image_get_width();
-		tsk->icon_height = imlib_image_get_height();
-		tsk->icon_data = malloc (tsk->icon_width * tsk->icon_height * sizeof (DATA32));
-		if (tsk->icon_data)
-			memcpy (tsk->icon_data, data, tsk->icon_width * tsk->icon_height * sizeof (DATA32));
-		if (imlib_context_get_image() == img)
-			imlib_free_image();
 		XFree(hints);
 	}
+	if (img == NULL) {
+		imlib_context_set_image(default_icon);
+		img = imlib_clone_image();
+	}
 
-	// resize, opacity and HSB
-/*
-	Imlib_Image icon;
-	Imlib_Color_Modifier cmod;
-	DATA8 red[256], green[256], blue[256], alpha[256];
-
-	icon = imlib_create_image_using_data (tsk->icon_width, tsk->icon_height, icon_data);
-	imlib_context_set_image (icon);
-	imlib_context_set_drawable (*pmap);
-
-	cmod = imlib_create_color_modifier ();
-	imlib_context_set_color_modifier (cmod);
-	imlib_image_set_has_alpha (1);
-	imlib_get_color_modifier_tables (red, green, blue, alpha);
-
-	int i, opacity;
-	opacity = (active == 0) ? (255*panel->g_task.font.alpha) : (255*panel->g_task.font_active.alpha);
-	for (i = 127; i < 256; i++) alpha[i] = opacity;
-
-	imlib_set_color_modifier_tables (red, green, blue, alpha);
-
-	//imlib_render_image_on_drawable (pos_x, pos_y);
-	imlib_render_image_on_drawable_at_size (pos_x, panel->g_task.icon_posy, panel->g_task.icon_size1, panel->g_task.icon_size1);
-
-	imlib_free_color_modifier ();
-	imlib_free_image ();
- */
-	if (tsk->icon_data) {
-		tsk->icon_data_active = malloc (tsk->icon_width * tsk->icon_height * sizeof (DATA32));
-		memcpy (tsk->icon_data_active, tsk->icon_data, tsk->icon_width * tsk->icon_height * sizeof (DATA32));
+	// transform icons
+	imlib_context_set_image(img);
+	tsk->icon = imlib_create_cropped_scaled_image(0, 0, imlib_image_get_width(), imlib_image_get_height(), panel->g_task.icon_size1, panel->g_task.icon_size1);
+	imlib_free_image();
+
+	imlib_context_set_image(tsk->icon);
+	tsk->icon_width = imlib_image_get_width();
+	tsk->icon_height = imlib_image_get_height();
+	tsk->icon_active = imlib_clone_image();
+
+	DATA32 *data32;
+	if (panel->g_task.alpha != 100 || panel->g_task.saturation != 0 || panel->g_task.brightness != 0) {
+		data32 = imlib_image_get_data();
+		adjust_asb(data32, tsk->icon_width, tsk->icon_height, panel->g_task.alpha, (float)panel->g_task.saturation/100, (float)panel->g_task.brightness/100);
+		imlib_image_put_back_data(data32);
+	}
 
-		if (panel->g_task.hue != 0 || panel->g_task.saturation != 0 || panel->g_task.brightness != 0) {
-			adjust_hsb(tsk->icon_data, tsk->icon_width, tsk->icon_height, (float)panel->g_task.hue/100, (float)panel->g_task.saturation/100, (float)panel->g_task.brightness/100);
-		}
-		if (panel->g_task.hue_active != 0 || panel->g_task.saturation_active != 0 || panel->g_task.brightness_active != 0) {
-			adjust_hsb(tsk->icon_data_active, tsk->icon_width, tsk->icon_height, (float)panel->g_task.hue_active/100, (float)panel->g_task.saturation_active/100, (float)panel->g_task.brightness_active/100);
-		}
+	if (panel->g_task.alpha_active != 100 || panel->g_task.saturation_active != 0 || panel->g_task.brightness_active != 0) {
+		imlib_context_set_image(tsk->icon_active);
+		data32 = imlib_image_get_data();
+		adjust_asb(data32, tsk->icon_width, tsk->icon_height, panel->g_task.alpha_active, (float)panel->g_task.saturation_active/100, (float)panel->g_task.brightness_active/100);
+		imlib_image_put_back_data(data32);
 	}
 }
 
 
 void draw_task_icon (Task *tsk, int text_width, int active)
 {
-	if (tsk->icon_data == 0 || tsk->icon_data_active == 0) return;
-
-	Pixmap *pmap = (active == 0) ? (&tsk->area.pix.pmap) : (&tsk->area.pix_active.pmap);
-	unsigned int *icon_data = (active == 0) ? (tsk->icon_data) : (tsk->icon_data_active);
+	if (tsk->icon == NULL || tsk->icon_active == NULL) return;
 
 	// Find pos
 	int pos_x;
@@ -299,30 +273,17 @@ void draw_task_icon (Task *tsk, int text_width, int active)
 	else pos_x = panel->g_task.area.paddingxlr + panel->g_task.area.pix.border.width;
 
 	// Render
-	Imlib_Image icon;
-	Imlib_Color_Modifier cmod;
-	DATA8 red[256], green[256], blue[256], alpha[256];
-
-	icon = imlib_create_image_using_data (tsk->icon_width, tsk->icon_height, icon_data);
-	imlib_context_set_image (icon);
+	Pixmap *pmap;
+	if (active == 0) {
+		imlib_context_set_image (tsk->icon);
+		pmap = &tsk->area.pix.pmap;
+	}
+	else {
+		imlib_context_set_image (tsk->icon_active);
+		pmap = &tsk->area.pix_active.pmap;
+	}
 	imlib_context_set_drawable (*pmap);
-
-	cmod = imlib_create_color_modifier ();
-	imlib_context_set_color_modifier (cmod);
-	imlib_image_set_has_alpha (1);
-	imlib_get_color_modifier_tables (red, green, blue, alpha);
-
-	int i, opacity;
-	opacity = (active == 0) ? (255*panel->g_task.font.alpha) : (255*panel->g_task.font_active.alpha);
-	for (i = 127; i < 256; i++) alpha[i] = opacity;
-
-	imlib_set_color_modifier_tables (red, green, blue, alpha);
-
-	//imlib_render_image_on_drawable (pos_x, pos_y);
-	imlib_render_image_on_drawable_at_size (pos_x, panel->g_task.icon_posy, panel->g_task.icon_size1, panel->g_task.icon_size1);
-
-	imlib_free_color_modifier ();
-	imlib_free_image ();
+	imlib_render_image_on_drawable (pos_x, panel->g_task.icon_posy);
 }
 
 
diff --git a/src/taskbar/task.h b/src/taskbar/task.h
index 172b0d5..981509e 100644
--- a/src/taskbar/task.h
+++ b/src/taskbar/task.h
@@ -26,8 +26,8 @@ typedef struct {
 	int icon_size1;
 	int maximum_width;
 	int maximum_height;
-	int hue, saturation, brightness;
-	int hue_active, saturation_active, brightness_active;
+	int alpha, saturation, brightness;
+	int alpha_active, saturation_active, brightness_active;
 	// starting position for text ~ task_padding + task_border + icon_size
 	double text_posx, text_posy;
 
@@ -46,9 +46,8 @@ typedef struct {
 	// TODO: group task with list of windows here
 	Window win;
 	int  desktop;
-	// ARGB icon
-	unsigned int *icon_data;
-	unsigned int *icon_data_active;
+	Imlib_Image icon;
+	Imlib_Image icon_active;
 	unsigned int icon_width;
 	unsigned int icon_height;
 	char *title;
diff --git a/src/tint.c b/src/tint.c
index e3fdb82..db63467 100644
--- a/src/tint.c
+++ b/src/tint.c
@@ -556,8 +556,8 @@ void event_property_notify (XEvent *e)
 						if (tsk->win == tsk2->win && tsk != tsk2) {
 							tsk2->icon_width = tsk->icon_width;
 							tsk2->icon_height = tsk->icon_height;
-							tsk2->icon_data = tsk->icon_data;
-							tsk2->icon_data_active = tsk->icon_data_active;
+							tsk2->icon = tsk->icon;
+							tsk2->icon_active = tsk->icon_active;
 							tsk2->area.redraw = 1;
 						}
 					}
diff --git a/src/util/common.c b/src/util/common.c
index f0f6b54..55a853b 100644
--- a/src/util/common.c
+++ b/src/util/common.c
@@ -30,7 +30,7 @@
 
 
 
-void adjust_hsb(DATA32 *data, int w, int h, float hu, float satur, float bright)
+void adjust_asb(DATA32 *data, int w, int h, int alpha, float satur, float bright)
 {
 	unsigned int x, y;
 	unsigned int a, r, g, b, argb;
@@ -44,6 +44,8 @@ void adjust_hsb(DATA32 *data, int w, int h, float hu, float satur, float bright)
 		for(id = y * w, x = 0; x < w; x++, id++) {
 			argb = data[id];
 			a = (argb >> 24) & 0xff;
+			// transparent => nothing to do.
+			if (a == 0) continue;
 			r = (argb >> 16) & 0xff;
 			g = (argb >> 8) & 0xff;
 			b = (argb) & 0xff;
@@ -82,9 +84,8 @@ void adjust_hsb(DATA32 *data, int w, int h, float hu, float satur, float bright)
 			brightness += bright;
 			if (brightness < 0.0) brightness = 0.0;
 			if (brightness > 1.0) brightness = 1.0;
-			hue += hu;
-			if (hue < 0.0) hue = 0.0;
-			if (hue > 1.0) hue = 1.0;
+			if (alpha != 100)
+				a = (a * alpha)/100;
 
 			// convert HSB to RGB
 			if (saturation == 0) {
@@ -138,4 +139,3 @@ void adjust_hsb(DATA32 *data, int w, int h, float hu, float satur, float bright)
 	}
 }
 
-
diff --git a/src/util/common.h b/src/util/common.h
index a2a0c76..fd22c55 100644
--- a/src/util/common.h
+++ b/src/util/common.h
@@ -50,9 +50,9 @@ typedef struct config_color
 } config_color;
 
 
-// adjust HSB on an ARGB icon
-void adjust_hsb(DATA32 *data, int w, int h, float hue, float satur, float bright);
-
+// adjust Alpha/Saturation/Brightness on an ARGB icon
+// alpha from 0 to 100, satur from 0 to 1, bright from 0 to 1.
+void adjust_asb(DATA32 *data, int w, int h, int alpha, float satur, float bright);
 
 
 #endif
-- 
2.44.0