[Opensync-commits] r5416 - trunk/opensync/common

[<svn...@op...>]

Author: dgollub
Date: Sun Mar 29 14:36:42 2009
New Revision: 5416
URL: http://www.opensync.org/changeset/5416

Log:
Tolerate NULL in osync_list_free and osync_list_free1 as list argument.
This avoids sanity checks when dealing with osync_list which is now more
common with the API list-pattern changes.

Modified:
   trunk/opensync/common/opensync_list.c
   trunk/opensync/common/opensync_list.h

Modified: trunk/opensync/common/opensync_list.c
==============================================================================
--- trunk/opensync/common/opensync_list.c	Sun Mar 29 00:02:50 2009	(r5415)
+++ trunk/opensync/common/opensync_list.c	Sun Mar 29 14:36:42 2009	(r5416)
@@ -47,12 +47,14 @@
 void
 osync_list_free (OSyncList *list)
 {
+	osync_return_if_fail(list);
 	g_slice_free_chain (OSyncList, list, next);
 }
 
 void
 osync_list_free_1 (OSyncList *list)
 {
+	osync_return_if_fail(list);
 	_osync_list_free1 (list);
 }
 

Modified: trunk/opensync/common/opensync_list.h
==============================================================================
--- trunk/opensync/common/opensync_list.h	Sun Mar 29 00:02:50 2009	(r5415)
+++ trunk/opensync/common/opensync_list.h	Sun Mar 29 14:36:42 2009	(r5416)
@@ -47,9 +47,15 @@
 /* Doubly linked lists
  */
 OSYNC_EXPORT OSyncList*   osync_list_alloc                   (void);
+
+/* @note: Slightly modified from original glib version.
+ * osync_list_free() and osync_list_free1 will immeditally return
+ * if list is NULL
+ */
 OSYNC_EXPORT void     osync_list_free                    (OSyncList            *list);
 OSYNC_EXPORT void     osync_list_free_1                  (OSyncList            *list);
 #define osync_list_free1                   osync_list_free_1
+
 OSYNC_EXPORT OSyncList*   osync_list_append                  (OSyncList            *list,
 					 void *          data);
 OSYNC_EXPORT OSyncList*   osync_list_prepend                 (OSyncList            *list,