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

[galago-commits at freedesktop.org]

Author: chipx86
Date: 2006-11-09 17:45:26 -0800 (Thu, 09 Nov 2006)
New Revision: 2948

Modified:
   trunk/notification-daemon/ChangeLog
   trunk/notification-daemon/src/themes/standard/theme.c
Log:
Instead of checking if the shape points array is NULL, just assert that it isn't. It should never be.


Modified: trunk/notification-daemon/ChangeLog
===================================================================
--- trunk/notification-daemon/ChangeLog	2006-11-10 00:50:22 UTC (rev 2947)
+++ trunk/notification-daemon/ChangeLog	2006-11-10 01:45:26 UTC (rev 2948)
@@ -1,3 +1,9 @@
+Thu Nov 09 17:45:01 PST 2006  Christian Hammond <chipx86 at chipx86.com>
+
+	* src/themes/standard/theme.c:
+	  - Instead of checking if the shape points array is NULL, just assert
+	    that it isn't. It should never be.
+
 Thu Nov 09 16:49:39 PST 2006  Christian Hammond <chipx86 at chipx86.com>
 
 	* src/themes/standard/theme.c:

Modified: trunk/notification-daemon/src/themes/standard/theme.c
===================================================================
--- trunk/notification-daemon/src/themes/standard/theme.c	2006-11-10 00:50:22 UTC (rev 2947)
+++ trunk/notification-daemon/src/themes/standard/theme.c	2006-11-10 01:45:26 UTC (rev 2948)
@@ -321,13 +321,12 @@
 			break;
 	}
 
-	if (shape_points != NULL)
-	{
-		windata->window_region =
-			gdk_region_polygon(shape_points, windata->num_border_points,
-							   GDK_EVEN_ODD_RULE);
-		g_free(shape_points);
-	}
+	g_assert(shape_points != NULL);
+
+	windata->window_region =
+		gdk_region_polygon(shape_points, windata->num_border_points,
+						   GDK_EVEN_ODD_RULE);
+	g_free(shape_points);
 }
 
 static void