On Tue, Jul 24, 2012 at 14:22:44 +0100, Daniel P. Berrange wrote:
From: "Daniel P. Berrange" <berrange(a)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(a)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