[Date Prev][Date Next]   [Thread Prev][Thread Next]   [Thread Index] [Date Index] [Author Index]

Re: [libvirt] [PATCH 14/17] * remote_protocol.c, remote_protocol.h: regenerate



On Tue, Oct 28, 2008 at 12:21:31PM +0100, Jim Meyering wrote:
> From: Jim Meyering <meyering redhat com>
> 
> ---
>  qemud/remote_protocol.c |    3 +++
>  qemud/remote_protocol.h |    2 ++
>  2 files changed, 5 insertions(+), 0 deletions(-)
> 
> diff --git a/qemud/remote_protocol.c b/qemud/remote_protocol.c
> index 06b33d8..652d5b5 100644
> --- a/qemud/remote_protocol.c
> +++ b/qemud/remote_protocol.c
> @@ -3,9 +3,12 @@
>   * It was generated using rpcgen.
>   */
> 
> +#include "remote_protocol.h"
>  #include <config.h>
>  #include "internal.h"
>  #include "socketcompat.h"
> +#include "remote_protocol.h"
> +#include <rpc/xdr.h>

This doesn't look right having the same file included twice.

> 
>  bool_t
>  xdr_remote_nonnull_string (XDR *xdrs, remote_nonnull_string *objp)
> diff --git a/qemud/remote_protocol.h b/qemud/remote_protocol.h
> index e43514d..bfcf40b 100644
> --- a/qemud/remote_protocol.h
> +++ b/qemud/remote_protocol.h
> @@ -16,6 +16,8 @@ extern "C" {
>  #include <config.h>
>  #include "internal.h"
>  #include "socketcompat.h"
> +#include "remote_protocol.h"
> +#include <rpc/xdr.h>

And including itself here.

Daniel
-- 
|: Red Hat, Engineering, London   -o-   http://people.redhat.com/berrange/ :|
|: http://libvirt.org  -o-  http://virt-manager.org  -o-  http://ovirt.org :|
|: http://autobuild.org       -o-         http://search.cpan.org/~danberr/ :|
|: GnuPG: 7D3B9505  -o-  F3C9 553F A1DA 4AC2 5648 23C1 B3DF F742 7D3B 9505 :|


[Date Prev][Date Next]   [Thread Prev][Thread Next]   [Thread Index] [Date Index] [Author Index]