[libvirt] [PATCH 24/30] openvz: Remove virConnectPtr from openvzError

Eric Blake eblake at redhat.com
Mon Apr 5 17:13:25 UTC 2010


On 04/04/2010 11:36 AM, Matthias Bolte wrote:
> ---
>  src/openvz/openvz_conf.c   |   83 +++++++-------
>  src/openvz/openvz_conf.h   |    9 +-
>  src/openvz/openvz_driver.c |  266 ++++++++++++++++++++++----------------------
>  3 files changed, 177 insertions(+), 181 deletions(-)
> 
> @@ -288,18 +286,18 @@ openvzReadNetworkConf(virConnectPtr conn,
>                      p += 4;
>                      len = next - p;
>                      if (len != 17) { //should be 17
> -                        openvzError(conn, VIR_ERR_INTERNAL_ERROR,
> -                              "%s", _("Wrong length MAC address"));
> +                        openvzError( VIR_ERR_INTERNAL_ERROR, "%s",
> +                                    _("Wrong length MAC address"));

Wonky spacing.

> +++ b/src/openvz/openvz_conf.h
> @@ -33,10 +33,8 @@
>  # include "domain_conf.h"
>  # include "threads.h"
>  
> -enum { OPENVZ_WARN, OPENVZ_ERR };

Worth mentioning in the commit log that these were unused?

> -
> -# define openvzError(conn, code, ...)                                       \
> -        virReportErrorHelper(conn, VIR_FROM_OPENVZ, code, __FILE__,        \
> +# define openvzError(code, ...)                                            \
> +        virReportErrorHelper(NULL, VIR_FROM_OPENVZ, code, __FILE__,        \
>                               __FUNCTION__, __LINE__, __VA_ARGS__)

ACK, and the rest of the patch is mechanical fallout, once you fix the nits.

-- 
Eric Blake   eblake at redhat.com    +1-801-349-2682
Libvirt virtualization library http://libvirt.org

-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 323 bytes
Desc: OpenPGP digital signature
URL: <http://listman.redhat.com/archives/libvir-list/attachments/20100405/cba2757c/attachment-0001.sig>


More information about the libvir-list mailing list