[Galago-commits] r2902 - in trunk/notification-daemon: . themes/standard

[galago-commits at freedesktop.org]

Author: chipx86
Date: 2006-09-07 10:32:08 -0700 (Thu, 07 Sep 2006)
New Revision: 2902

Modified:
   trunk/notification-daemon/ChangeLog
   trunk/notification-daemon/NEWS
   trunk/notification-daemon/themes/standard/theme.c
Log:
Patch by Nick Schermer to fix a memory leak. We lost 4 bytes for every notification displayed with the standard theme.


Modified: trunk/notification-daemon/ChangeLog
===================================================================
--- trunk/notification-daemon/ChangeLog	2006-09-05 07:04:01 UTC (rev 2901)
+++ trunk/notification-daemon/ChangeLog	2006-09-07 17:32:08 UTC (rev 2902)
@@ -1,3 +1,9 @@
+Thu Sep 07 10:28:50 PDT 2006  Christian Hammond <chipx86 at chipx86.com>
+
+	* themes/standard/theme.c:
+	  - Patch by Nick Schermer to fix a memory leak. We lost 4 bytes for
+	    every notification displayed with the standard theme.
+
 Sun Jul 30 15:32:10 PDT 2006  Christian Hammond <chipx86 at chipx86.com>
 
 	* src/daemon.c:

Modified: trunk/notification-daemon/NEWS
===================================================================
--- trunk/notification-daemon/NEWS	2006-09-05 07:04:01 UTC (rev 2901)
+++ trunk/notification-daemon/NEWS	2006-09-07 17:32:08 UTC (rev 2902)
@@ -16,6 +16,8 @@
 	  notification would cause a broken notification where timeouts and
 	  action invoking fails. In this case, a new ID is generated. Patch by
 	  Ed Catmur. (Bug #76)
+	* Fixed a memory leak that occurred every time a notification went away.
+	  Patch by Nick Schermer.
 
 version 0.3.5 (26-April-2006):
 	* Add three new required theme functions:

Modified: trunk/notification-daemon/themes/standard/theme.c
===================================================================
--- trunk/notification-daemon/themes/standard/theme.c	2006-09-05 07:04:01 UTC (rev 2901)
+++ trunk/notification-daemon/themes/standard/theme.c	2006-09-07 17:32:08 UTC (rev 2902)
@@ -162,6 +162,8 @@
 
 	if (windata->window_region != NULL)
 		gdk_region_destroy(windata->window_region);
+
+	g_free(windata);
 }
 
 static void