diff src/cache-loader.c @ 1523:24a12aa0cb54

Fix up event source ids type: gint -> guint. Functions like g_timeout_add() or g_idle_add() return a guint greater than 0, but in most places it was wrongly stored as int and initialized to -1. This broke assertions matching in g_source_remove() for example since id was always greater than 0 even when timer was not set (-1 was casted to the biggest guint).

--- a/src/cache-loader.c	Mon Apr 06 21:52:49 2009 +0000
+++ b/src/cache-loader.c	Mon Apr 06 22:13:54 2009 +0000
@@ -174,7 +174,7 @@
 			g_free(base);
 			}
 
-		cl->idle_id = -1;
+		cl->idle_id = 0;
 
 		if (cl->done_func)
 			{
@@ -232,10 +232,10 @@
 {
 	if (!cl) return;
 
-	if (cl->idle_id != -1)
+	if (cl->idle_id)
 		{
 		g_source_remove(cl->idle_id);
-		cl->idle_id = -1;
+		cl->idle_id = 0;
 		}
 
 	image_loader_free(cl->il);