[libvirt] [PATCH v2] parallels: minor cleanup

Michal Privoznik mprivozn at redhat.com
Tue Mar 17 15:56:48 UTC 2015


On 17.03.2015 16:21, Maxim Nestratov wrote:
> v2 change:
>     rebased to the current master

We usually put the diff to previous versions in between commit message
and diff stat. That way git ignores the diff and it's not part of the
commit message.

> 
> indentation is fixed, unnecessary error message removed,
> unnecessary job freeing removed

This is not much explanatory.

> 
> Signed-off-by: Maxim Nestratov <mnestratov at parallels.com>
> ---
>  src/parallels/parallels_sdk.c |    5 +----
>  1 files changed, 1 insertions(+), 4 deletions(-)
> 
> diff --git a/src/parallels/parallels_sdk.c b/src/parallels/parallels_sdk.c
> index ee71bd6..a0b62cd 100644
> --- a/src/parallels/parallels_sdk.c
> +++ b/src/parallels/parallels_sdk.c
> @@ -701,7 +701,7 @@ prlsdkGetNetInfo(PRL_HANDLE netAdapter, virDomainNetDefPtr net, bool isCt)
>          goto cleanup;
>  
>      pret = PrlVmDevNet_GetHostInterfaceName(netAdapter, net->ifname, &buflen);
> -        prlsdkCheckRetGoto(pret, cleanup);
> +    prlsdkCheckRetGoto(pret, cleanup);
>  
>      if (isCt && netAdapterIndex == (PRL_UINT32) -1) {
>          /* venet devices don't have mac address and
> @@ -1350,7 +1350,6 @@ prlsdkLoadDomains(parallelsConnPtr privconn)
>  
>   error:
>      PrlHandle_Free(result);
> -    PrlHandle_Free(job);
>      return -1;
>  }
>  
> @@ -1730,8 +1729,6 @@ prlsdkDomainChangeState(virDomainPtr domain,
>  
>      pdom = dom->privateData;
>      pret = chstate(privconn, pdom->sdkdom);
> -    virReportError(VIR_ERR_OPERATION_FAILED,
> -                   _("Can't change domain state: %d"), pret);
>      if (PRL_FAILED(pret)) {
>          virResetLastError();
>  
> 

I've updated the commit message, ACKed and pushed.

Michal




More information about the libvir-list mailing list