
δΊ 2013-3-22 2:33, John Ferlan ει:
On 03/20/2013 11:39 PM, Wenchao Xia wrote:
This is a pure code move patch. Now codes using libvirt event or native event, are moved into one macro protection, which make code easy to read, and in futher they can be moved into new file as CSI-libvirt.c. This patch also fix code style problem in moved code.
Signed-off-by: Wenchao Xia <xiawenc@linux.vnet.ibm.com> --- src/Virt_ComputerSystemIndication.c | 851 ++++++++++++++++++----------------- 1 files changed, 429 insertions(+), 422 deletions(-)
ACK - lots of trust here as it's not fun looking around for code movement :-)
Personally, I prefer interlaced code as it's somewhat easier to maintain long term. Now there's lots duplicated (eg, cut-n-paste) code that's can be the source of future "issues" when one stream is changed but not the other.
Yes, that would be a problem. Duplicated code is not many now(activate/deactite, etc), we can adjust them later.
John
_______________________________________________ Libvirt-cim mailing list Libvirt-cim@redhat.com https://www.redhat.com/mailman/listinfo/libvirt-cim
-- Best Regards Wenchao Xia