On Thu, Jun 23, 2016 at 10:47:25PM +0200, Peter Krempa wrote:
On Thu, Jun 23, 2016 at 22:38:46 +0200, Ján Tomko wrote:
> Add SASL at the end to make the format obvious.
> ---
> src/libvirt_remote.syms | 4 ++--
> src/remote/remote_driver.c | 4 ++--
> src/rpc/virnetclient.c | 4 ++--
> src/rpc/virnetclient.h | 4 ++--
> 4 files changed, 8 insertions(+), 8 deletions(-)
>
> diff --git a/src/rpc/virnetclient.c b/src/rpc/virnetclient.c
> index c43cd08..361dc1a 100644
> --- a/src/rpc/virnetclient.c
> +++ b/src/rpc/virnetclient.c
> @@ -944,12 +944,12 @@ void virNetClientRemoveStream(virNetClientPtr client,
> }
>
>
> -const char *virNetClientLocalAddrString(virNetClientPtr client)
> +const char *virNetClientLocalAddrStringSASL(virNetClientPtr client)
> {
> return virNetSocketLocalAddrStringSASL(client->sock);
> }
Is it even worth having such thin wrappers?
Well they let us hide the internals of virNetClient and virNetSocket
structures.
ACK
Thanks, pushed.
Jan