[Opensync-commits] r5084 - trunk/opensync/data

[<dg...@su...>]

Author: dgollub
Date: Sun Jan 11 00:40:36 2009
New Revision: 5084
URL: http://www.opensync.org/changeset/5084

Log:
Instrument osync_change_compare() with sensitive tracing

Modified:
   trunk/opensync/data/opensync_change.c

Modified: trunk/opensync/data/opensync_change.c
==============================================================================
--- trunk/opensync/data/opensync_change.c	Sat Jan 10 23:33:46 2009	(r5083)
+++ trunk/opensync/data/opensync_change.c	Sun Jan 11 00:40:36 2009	(r5084)
@@ -184,7 +184,33 @@
 	osync_assert(leftchange);
 
 	if (rightchange->changetype == leftchange->changetype) {
-		OSyncConvCmpResult ret = osync_data_compare(leftchange->data, rightchange->data);
+		OSyncConvCmpResult ret;
+
+		if (osync_trace_is_enabled()) {
+			char *leftprint, *rightprint;
+			
+			leftprint = osync_data_get_printable(leftchange->data);
+			rightprint = osync_data_get_printable(rightchange->data);
+
+			osync_trace(TRACE_SENSITIVE, "\nleft change (UID:%s):\n%s\n"
+				"right change (UID:%s):\n%s\n",
+				__NULLSTR(osync_change_get_uid(leftchange)),
+				__NULLSTR(leftprint),
+				__NULLSTR(osync_change_get_uid(rightchange)),
+				__NULLSTR(rightprint));
+
+			/* FIXME: free the result of objformat_print(). There is
+			 * no object format specific destory function for print?
+			 */
+
+			if (leftprint)
+				osync_free(leftprint);
+
+			if (rightprint)
+				osync_free(rightprint);
+		}
+
+		ret = osync_data_compare(leftchange->data, rightchange->data);
 		osync_trace(TRACE_EXIT, "%s: Compare data: %i", __func__, ret);
 		return ret;
 	} else {