Jay Gagnon wrote:
}
+ for (i = 0; i < prev_count; i++) { + res = dom_changed(prev_xml[i], cur_xml, cur_count); + if (res) { + CU_DEBUG("Domain '%s' modified.", prev_xml[i].uuid); + mod_ind(context, conn, prev_xml[i], prefix, ns); + } + free_dom_xml(prev_xml[i]); + }
Would it be worth while to add the contents of this loop with delete indication loop just above? for (i = 0; i < prev_count; i++) { //Handle delete indication } This would prevent us from looping through another time. However, it might make the loop too complex / hard to read... -- Kaitlin Rupert IBM Linux Technology Center kaitlin@linux.vnet.ibm.com