project_files/frontlib/model/roomlist.c
changeset 7316 f7b49b2c5d84
parent 7314 6171f0bad318
--- a/project_files/frontlib/model/roomlist.c	Wed Jun 27 22:52:19 2012 +0200
+++ b/project_files/frontlib/model/roomlist.c	Thu Jul 05 00:33:24 2012 +0200
@@ -86,9 +86,7 @@
 
 int flib_roomlist_add(flib_roomlist *list, char **params) {
 	int result = -1;
-	if(!list || !params) {
-		flib_log_e("null parameter in flib_roomlist_add");
-	} else {
+	if(!log_badargs_if2(list==NULL, params==NULL)) {
 		flib_room *tmpRoom = fillRoomFromParams(params);
 		if(tmpRoom) {
 			if(!insertRoom(&list->rooms, &list->roomCount, tmpRoom, 0)) {
@@ -103,9 +101,7 @@
 
 int flib_roomlist_delete(flib_roomlist *list, const char *name) {
 	int result = -1;
-	if(!list || !name) {
-		flib_log_e("null parameter in flib_roomlist_delete");
-	} else {
+	if(!log_badargs_if2(list==NULL, name==NULL)) {
 		int roomid = findRoom(list, name);
 		if(roomid<0) {
 			flib_log_w("Attempt to delete unknown room %s", name);
@@ -122,9 +118,7 @@
 
 int flib_roomlist_update(flib_roomlist *list, const char *name, char **params) {
 	int result = -1;
-	if(!list || !name || !params) {
-		flib_log_e("null parameter in flib_roomlist_update");
-	} else {
+	if(!log_badargs_if3(list==NULL, name==NULL, params==NULL)) {
 		flib_room *tmpRoom = fillRoomFromParams(params);
 		int roomid = findRoom(list, name);
 		if(tmpRoom && roomid>=0) {
@@ -140,9 +134,7 @@
 
 flib_room *flib_roomlist_find(const flib_roomlist *list, const char *name) {
 	flib_room *result = NULL;
-	if(!list || !name) {
-		flib_log_e("null parameter in flib_roomlist_find");
-	} else {
+	if(!log_badargs_if2(list==NULL, name==NULL)) {
 		int roomid = findRoom(list, name);
 		if(roomid>=0) {
 			result = list->rooms[roomid];
@@ -152,9 +144,7 @@
 }
 
 void flib_roomlist_clear(flib_roomlist *list) {
-	if(!list) {
-		flib_log_e("null parameter in flib_roomlist_clear");
-	} else {
+	if(!log_badargs_if(list==NULL)) {
 		for(int i=0; i<list->roomCount; i++) {
 			flib_roomlist_room_destroy(list->rooms[i]);
 		}