[libvirt] [PATCH python] Fix calling of virStreamSend method

Osier Yang jyang at redhat.com
Thu Jan 23 14:47:08 UTC 2014


On 23/01/14 22:25, Daniel P. Berrange wrote:

About the subject prefix, [libvirt-python] should be better, like
what we did for Perl binding.

> Change d40861 removed the 'len' argument from the virStreamSend
> C level wrapper, but forgot to remove it from the python level
> wrapper.
>
> Reported-by: Robie Basak <robie.basak at canonical.com>
> Signed-off-by: Daniel P. Berrange <berrange at redhat.com>
> ---
>   libvirt-override-virStream.py | 2 +-
>   1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/libvirt-override-virStream.py b/libvirt-override-virStream.py
> index cd44314..ce82da6 100644
> --- a/libvirt-override-virStream.py
> +++ b/libvirt-override-virStream.py
> @@ -122,6 +122,6 @@
>           with the call, but may instead be delayed until a
>           subsequent call.
>           """
> -        ret = libvirtmod.virStreamSend(self._o, data, len(data))
> +        ret = libvirtmod.virStreamSend(self._o, data)
>           if ret == -1: raise libvirtError ('virStreamSend() failed')
>           return ret
ACK




More information about the libvir-list mailing list