On Tue, Jul 24, 2012 at 14:22:44 +0100, Daniel P. Berrange wrote:
> From: "Daniel P. Berrange" <berrange@xxxxxxxxxx>
>
> 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@xxxxxxxxxx>
> ---
> 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
--
libvir-list mailing list
libvir-list@xxxxxxxxxx
https://www.redhat.com/mailman/listinfo/libvir-list