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 <[email protected]>
> ---
>  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.

John

_______________________________________________
Libvirt-cim mailing list
[email protected]
https://www.redhat.com/mailman/listinfo/libvirt-cim

Reply via email to