Add more regression test and fix Integer and Real type handling.

1 files changed, 67 insertions, 0 deletions

diff --git a/src/plist.c b/src/plist.c
index 0e73c4b..e800b5c 100644
--- a/src/plist.c
+++ b/src/plist.c
@@ -414,3 +414,70 @@ void plist_get_date_val(plist_t node, int32_t * sec, int32_t * usec)
 	*sec = val.tv_sec;
 	*usec = val.tv_usec;
 }
+
+gboolean plist_data_compare(gconstpointer a, gconstpointer b)
+{
+	plist_data_t val_a = NULL;
+	plist_data_t val_b = NULL;
+
+	if (!a || !b)
+		return FALSE;
+
+	if (!((GNode *) a)->data || !((GNode *) b)->data)
+		return FALSE;
+
+	val_a = plist_get_data((plist_t) a);
+	val_b = plist_get_data((plist_t) b);
+
+	if (val_a->type != val_b->type)
+		return FALSE;
+
+	switch (val_a->type) {
+	case PLIST_BOOLEAN:
+	case PLIST_UINT:
+	case PLIST_REAL:
+		if (val_a->intval == val_b->intval)	//it is an union so this is sufficient
+			return TRUE;
+		else
+			return FALSE;
+
+	case PLIST_KEY:
+	case PLIST_STRING:
+		if (!strcmp(val_a->strval, val_b->strval))
+			return TRUE;
+		else
+			return FALSE;
+	case PLIST_UNICODE:
+		if (!memcmp(val_a->unicodeval, val_b->unicodeval, val_a->length))
+			return TRUE;
+		else
+			return FALSE;
+
+	case PLIST_DATA:
+		if (!memcmp(val_a->buff, val_b->buff, val_a->length))
+			return TRUE;
+		else
+			return FALSE;
+	case PLIST_ARRAY:
+	case PLIST_DICT:
+		//compare pointer
+		if (a == b)
+			return TRUE;
+		else
+			return FALSE;
+		break;
+	case PLIST_DATE:
+		if (!memcmp(&(val_a->timeval), &(val_b->timeval), sizeof(GTimeVal)))
+			return TRUE;
+		else
+			return FALSE;
+	default:
+		break;
+	}
+	return FALSE;
+}
+
+char plist_compare_node_value(plist_t node_l, plist_t node_r)
+{
+	return plist_data_compare( node_l , node_r );
+}