Enlightenment CVS committal Author : sebastid Project : e17 Module : libs/ecore
Dir : e17/libs/ecore/src/lib/ecore Modified Files: ecore_list.c ecore_hash.c ecore_tree.c Log Message: Return NULL, not FALSE for pointers. =================================================================== RCS file: /cvsroot/enlightenment/e17/libs/ecore/src/lib/ecore/ecore_list.c,v retrieving revision 1.19 retrieving revision 1.20 diff -u -3 -r1.19 -r1.20 --- ecore_list.c 6 Sep 2005 19:29:24 -0000 1.19 +++ ecore_list.c 6 Sep 2005 21:00:55 -0000 1.20 @@ -494,7 +494,7 @@ { void *ret; - CHECK_PARAM_POINTER_RETURN("list", list, FALSE); + CHECK_PARAM_POINTER_RETURN("list", list, NULL); ret = _ecore_list_remove_0(list); @@ -508,13 +508,13 @@ Ecore_List_Node *old; if (!list) - return FALSE; + return NULL; if (ecore_list_is_empty(list)) - return FALSE; + return NULL; if (!list->current) - return FALSE; + return NULL; if (list->current == list->first) return _ecore_list_remove_first(list); @@ -569,7 +569,7 @@ { void *ret; - CHECK_PARAM_POINTER_RETURN("list", list, FALSE); + CHECK_PARAM_POINTER_RETURN("list", list, NULL); ret = _ecore_list_remove_first(list); @@ -583,13 +583,13 @@ Ecore_List_Node *old; if (!list) - return FALSE; + return NULL; if (ecore_list_is_empty(list)) - return FALSE; + return NULL; if (!list->first) - return FALSE; + return NULL; old = list->first; @@ -622,7 +622,7 @@ { void *ret; - CHECK_PARAM_POINTER_RETURN("list", list, FALSE); + CHECK_PARAM_POINTER_RETURN("list", list, NULL); ret = _ecore_list_remove_last(list); @@ -636,13 +636,13 @@ Ecore_List_Node *old, *prev; if (!list) - return FALSE; + return NULL; if (ecore_list_is_empty(list)) - return FALSE; + return NULL; if (!list->last) - return FALSE; + return NULL; old = list->last; if (list->current == old) @@ -689,7 +689,7 @@ { void *ret; - CHECK_PARAM_POINTER_RETURN("list", list, FALSE); + CHECK_PARAM_POINTER_RETURN("list", list, NULL); ret = _ecore_list_goto_index(list, index); @@ -703,20 +703,20 @@ int i; if (!list) - return FALSE; + return NULL; if (ecore_list_is_empty(list)) - return FALSE; + return NULL; if (index > ecore_list_nodes(list) || index < 0) - return FALSE; + return NULL; _ecore_list_goto_first(list); for (i = 0; i < index && _ecore_list_next(list); i++); if (i >= list->nodes) - return FALSE; + return NULL; list->index = i; @@ -734,7 +734,7 @@ { void *ret; - CHECK_PARAM_POINTER_RETURN("list", list, FALSE); + CHECK_PARAM_POINTER_RETURN("list", list, NULL); ret = _ecore_list_goto(list, data); @@ -785,7 +785,7 @@ { void *ret; - CHECK_PARAM_POINTER_RETURN("list", list, FALSE); + CHECK_PARAM_POINTER_RETURN("list", list, NULL); ret = _ecore_list_goto_first(list); @@ -814,7 +814,7 @@ { void *ret; - CHECK_PARAM_POINTER_RETURN("list", list, FALSE); + CHECK_PARAM_POINTER_RETURN("list", list, NULL); ret = _ecore_list_goto_last(list); @@ -870,7 +870,7 @@ { void *data; - CHECK_PARAM_POINTER_RETURN("list", list, FALSE); + CHECK_PARAM_POINTER_RETURN("list", list, NULL); data = _ecore_list_next(list); @@ -1224,7 +1224,7 @@ Ecore_List *l2 = ECORE_LIST(list); Ecore_DList_Node *node; - CHECK_PARAM_POINTER_RETURN("list", list, FALSE); + CHECK_PARAM_POINTER_RETURN("list", list, NULL); if (l2->current) { node = ECORE_DLIST_NODE(list->current->next); @@ -1246,7 +1246,7 @@ { void *ret; - CHECK_PARAM_POINTER_RETURN("list", list, FALSE); + CHECK_PARAM_POINTER_RETURN("list", list, NULL); ret = _ecore_dlist_remove_first(list); @@ -1281,7 +1281,7 @@ void *ret; if (!list) - return FALSE; + return NULL; ret = _ecore_list_remove_first(list); if (ret && ECORE_LIST(list)->first) @@ -1300,7 +1300,7 @@ { void *ret; - CHECK_PARAM_POINTER_RETURN("list", list, FALSE); + CHECK_PARAM_POINTER_RETURN("list", list, NULL); ret = _ecore_list_remove_last(list); @@ -1317,7 +1317,7 @@ { void *ret; - CHECK_PARAM_POINTER_RETURN("list", list, FALSE); + CHECK_PARAM_POINTER_RETURN("list", list, NULL); ret = _ecore_dlist_goto_index(list, index); @@ -1331,13 +1331,13 @@ int i, increment; if (!list) - return FALSE; + return NULL; if (ecore_list_is_empty(ECORE_LIST(list))) - return FALSE; + return NULL; if (index > ecore_list_nodes(ECORE_LIST(list)) || index < 0) - return FALSE; + return NULL; if (ECORE_LIST(list)->index >= ECORE_LIST(list)->nodes) _ecore_list_goto_last(ECORE_LIST(list)); @@ -1368,7 +1368,7 @@ { void *ret; - CHECK_PARAM_POINTER_RETURN("list", list, FALSE); + CHECK_PARAM_POINTER_RETURN("list", list, NULL); ret = _ecore_list_goto(ECORE_LIST(list), data); @@ -1385,7 +1385,7 @@ { void *ret; - CHECK_PARAM_POINTER_RETURN("list", list, FALSE); + CHECK_PARAM_POINTER_RETURN("list", list, NULL); ret = _ecore_list_goto_first(list); @@ -1401,7 +1401,7 @@ { void *ret; - CHECK_PARAM_POINTER_RETURN("list", list, FALSE); + CHECK_PARAM_POINTER_RETURN("list", list, NULL); ret = _ecore_list_goto_last(ECORE_LIST(list)); =================================================================== RCS file: /cvsroot/enlightenment/e17/libs/ecore/src/lib/ecore/ecore_hash.c,v retrieving revision 1.15 retrieving revision 1.16 diff -u -3 -r1.15 -r1.16 --- ecore_hash.c 6 Sep 2005 19:29:24 -0000 1.15 +++ ecore_hash.c 6 Sep 2005 21:00:55 -0000 1.16 @@ -356,7 +356,7 @@ void *data; Ecore_Hash_Node *node; - CHECK_PARAM_POINTER_RETURN("hash", hash, FALSE); + CHECK_PARAM_POINTER_RETURN("hash", hash, NULL); node = _ecore_hash_get_node(hash, key); if (!node) =================================================================== RCS file: /cvsroot/enlightenment/e17/libs/ecore/src/lib/ecore/ecore_tree.c,v retrieving revision 1.6 retrieving revision 1.7 diff -u -3 -r1.6 -r1.7 --- ecore_tree.c 6 Sep 2005 19:29:24 -0000 1.6 +++ ecore_tree.c 6 Sep 2005 21:00:55 -0000 1.7 @@ -175,7 +175,7 @@ { void *ret; - CHECK_PARAM_POINTER_RETURN("node", node, FALSE); + CHECK_PARAM_POINTER_RETURN("node", node, NULL); ECORE_READ_LOCK(node); ret = node->value; ECORE_READ_UNLOCK(node); @@ -210,7 +210,7 @@ { void *ret; - CHECK_PARAM_POINTER_RETURN("node", node, FALSE); + CHECK_PARAM_POINTER_RETURN("node", node, NULL); ECORE_READ_LOCK(node); ret = node->key; ECORE_READ_UNLOCK(node); @@ -254,7 +254,7 @@ { Ecore_Tree_Node *ret; - CHECK_PARAM_POINTER_RETURN("tree", tree, FALSE); + CHECK_PARAM_POINTER_RETURN("tree", tree, NULL); ECORE_READ_LOCK(tree); ret = tree_node_find(tree, key); @@ -274,7 +274,7 @@ void *ret; Ecore_Tree_Node *node; - CHECK_PARAM_POINTER_RETURN("tree", tree, FALSE); + CHECK_PARAM_POINTER_RETURN("tree", tree, NULL); ECORE_READ_LOCK(tree); node = tree_node_find(tree, key); @@ -298,7 +298,7 @@ { Ecore_Tree_Node *node; - CHECK_PARAM_POINTER_RETURN("tree", tree, FALSE); + CHECK_PARAM_POINTER_RETURN("tree", tree, NULL); ECORE_READ_LOCK(tree); node = tree_node_find(tree, key); @@ -334,7 +334,7 @@ { Ecore_Tree_Node *node; - CHECK_PARAM_POINTER_RETURN("tree", tree, FALSE); + CHECK_PARAM_POINTER_RETURN("tree", tree, NULL); ECORE_READ_LOCK(tree); node = tree_node_find(tree, key); @@ -614,7 +614,7 @@ { Ecore_Tree_Node *parent, *travel; - CHECK_PARAM_POINTER_RETURN("tree", tree, FALSE); + CHECK_PARAM_POINTER_RETURN("tree", tree, NULL); parent = tree_node_find(tree, key); if (parent) @@ -647,7 +647,7 @@ int compare; Ecore_Tree_Node *node; - CHECK_PARAM_POINTER_RETURN("tree", tree, FALSE); + CHECK_PARAM_POINTER_RETURN("tree", tree, NULL); node = tree->tree; while (node && (compare = tree->compare_func(key, node->key)) != 0) { ------------------------------------------------------- SF.Net email is Sponsored by the Better Software Conference & EXPO September 19-22, 2005 * San Francisco, CA * Development Lifecycle Practices Agile & Plan-Driven Development * Managing Projects & Teams * Testing & QA Security * Process Improvement & Measurement * http://www.sqe.com/bsce5sf _______________________________________________ enlightenment-cvs mailing list enlightenment-cvs@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/enlightenment-cvs