Re: [libvirt] [PATCH v2 0/2] Add error handling to optional arguments in cmdCPUStats

2013-04-11 Thread John Ferlan
Thanks for the review - adjusted/removed the -cpu value check w/r/t: patch 1/2 - every other structure in the module didn't have the trailing comma so that was a consistency thing. pushed John -- libvir-list mailing list libvir-list@redhat.com https://www.redhat.com/mailman/listinfo/libvir-list

[libvirt] [PATCH v2 0/2] Add error handling to optional arguments in cmdCPUStats

2013-04-08 Thread John Ferlan
https://bugzilla.redhat.com/show_bug.cgi?id=907732 Also added informational message when count value is larger than number of CPUs present. Original code commit '31047e2b' quietly changes it and continues on. Prior to patch 2/2, no errors were seen for following s