Mercurial > pidgin3 / changeset

• changeset
• raw

changeset

Fix two compile warnings. Did I do that? I totally didn't notice. My bad.

Fri, 26 Aug 2011 05:00:46 +0000

author

Mark Doliner <markdoliner@pidgin.im>

date

Fri, 26 Aug 2011 05:00:46 +0000

changeset 32237

d5398bd73913

parent 32236

b5a6bc53b7e6

child 32238

00bf2611f41c

Fix two compile warnings. Did I do that? I totally didn't notice. My bad.

roomlist.c: In function ‘purple_roomlist_room_get_expanded_once’:
roomlist.c:402:2: warning: return makes integer from pointer without a cast
roomlist.c: In function ‘purple_roomlist_room_set_expanded_once’:
roomlist.c:409:2: warning: ‘return’ with a value, in function returning void

--- a/libpurple/roomlist.c	Thu Aug 25 07:36:54 2011 +0000
+++ b/libpurple/roomlist.c	Fri Aug 26 05:00:46 2011 +0000
@@ -399,14 +399,14 @@
 
 gboolean purple_roomlist_room_get_expanded_once(PurpleRoomlistRoom *room)
 {
-	g_return_val_if_fail(room != NULL, NULL);
+	g_return_val_if_fail(room != NULL, FALSE);
 
 	return room->expanded_once;
 }
 
 void purple_roomlist_room_set_expanded_once(PurpleRoomlistRoom *room, gboolean expanded_once)
 {
-	g_return_val_if_fail(room != NULL, NULL);
+	g_return_if_fail(room != NULL);
 
 	room->expanded_once = expanded_once;
 }