|
From: <svn...@op...> - 2009-11-05 19:10:14
|
Author: dgollub Date: Thu Nov 5 20:10:05 2009 New Revision: 5939 URL: http://www.opensync.org/changeset/5939 Log: Fix osync_trace() varg list which was missing the error parameter. Added input_data and input_size of converter trace instrumentation for easier debugging. Modified: trunk/opensync/data/opensync_change.c trunk/opensync/format/opensync_converter.c Modified: trunk/opensync/data/opensync_change.c ============================================================================== --- trunk/opensync/data/opensync_change.c Thu Nov 5 20:09:09 2009 (r5938) +++ trunk/opensync/data/opensync_change.c Thu Nov 5 20:10:05 2009 (r5939) @@ -179,7 +179,7 @@ OSyncConvCmpResult osync_change_compare(OSyncChange *leftchange, OSyncChange *rightchange, OSyncError **error) { - osync_trace(TRACE_ENTRY, "%s(%p, %p)", __func__, leftchange, rightchange, error); + osync_trace(TRACE_ENTRY, "%s(%p, %p, %p)", __func__, leftchange, rightchange, error); osync_assert(rightchange); osync_assert(leftchange); Modified: trunk/opensync/format/opensync_converter.c ============================================================================== --- trunk/opensync/format/opensync_converter.c Thu Nov 5 20:09:09 2009 (r5938) +++ trunk/opensync/format/opensync_converter.c Thu Nov 5 20:10:05 2009 (r5939) @@ -190,7 +190,7 @@ osync_assert(converter->convert_func); /* Invoke the converter */ - osync_trace(TRACE_ENTRY, "Converter function from \"%s\" to \"%s\"", osync_objformat_get_name(converter->source_format), osync_objformat_get_name(converter->target_format)); + osync_trace(TRACE_ENTRY, "Converter function from \"%s\" to \"%s\" - input_data: %p input_size: %u", osync_objformat_get_name(converter->source_format), osync_objformat_get_name(converter->target_format), input_data, input_size); if (!converter->convert_func(input_data, input_size, &output_data, &output_size, &free_input, config, converter->userdata, error)) { osync_trace(TRACE_EXIT_ERROR, "Converter function: %s", osync_error_print(error)); |