[Libvirt-cim] [PATCH 1 of 2] Add Modified to the list of available indications in ComputerSystemIndication

[Kaitlin Rupert]

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 at linux.vnet.ibm.com