On Tue, Jul 24, 2012 at 14:22:44 +0100, Daniel P. Berrange wrote:
From: "Daniel P. Berrange" <berrange@redhat.com>
For consistency all the APIs in the lxc_process.c file should have a virLXCProcess prefix in their name
Signed-off-by: Daniel P. Berrange <berrange@redhat.com> --- src/lxc/lxc_driver.c | 24 +++--- src/lxc/lxc_process.c | 216 ++++++++++++++++++++++++------------------------- src/lxc/lxc_process.h | 41 +++++----- 3 files changed, 141 insertions(+), 140 deletions(-)
...
diff --git a/src/lxc/lxc_process.c b/src/lxc/lxc_process.c index ab27cbe..c25a04b 100644 --- a/src/lxc/lxc_process.c +++ b/src/lxc/lxc_process.c ... @@ -56,16 +56,16 @@ int lxcProcessAutoDestroyInit(lxc_driver_t *driver) return 0; }
-struct lxcProcessAutoDestroyData { +struct virLXCProcessAutoDestroyData { lxc_driver_t *driver; virConnectPtr conn; }; ... @@ -1114,25 +1114,25 @@ cleanup:
error: err = virSaveLastError(); - lxcVmTerminate(driver, vm, VIR_DOMAIN_SHUTOFF_FAILED); + virLXCProcessStop(driver, vm, VIR_DOMAIN_SHUTOFF_FAILED); goto cleanup; }
-struct lxcAutostartData { +struct virLXCAutostartData {
This should be virLXCProcessAutostartData for consistency.
lxc_driver_t *driver; virConnectPtr conn; };
... ACK Jirka