[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH 1/3] vhost: simple clean up
From: |
Sahid Orentino Ferdjaoui |
Subject: |
Re: [Qemu-devel] [PATCH 1/3] vhost: simple clean up |
Date: |
Fri, 7 Apr 2017 10:38:49 +0200 |
User-agent: |
Mutt/1.8.0 (2017-02-23) |
On Fri, Apr 07, 2017 at 02:58:46PM +0800, weiqi wrote:
> From: Wei Qi <address@hidden>
>
> It is unnecessary to assign 'hdev->nvqs' to 'i', because 'hdev->nvqs' unused
> after assignment, so remove it.
nit: The line should not be longer than 76 characters [0].
[0]
http://wiki.qemu-project.org/Contribute/SubmitAPatch#Write_a_meaningful_commit_message
>
> Signed-off-by: Wei Qi <address@hidden>
> ---
> hw/virtio/vhost.c | 1 -
> 1 file changed, 1 deletion(-)
Reviewed-By: Sahid Orentino Ferdjaoui <address@hidden>
>
> diff --git a/hw/virtio/vhost.c b/hw/virtio/vhost.c
> index 613494d..5c57983 100644
> --- a/hw/virtio/vhost.c
> +++ b/hw/virtio/vhost.c
> @@ -1557,7 +1557,6 @@ fail_vq:
> hdev->vqs + i,
> hdev->vq_index + i);
> }
> - i = hdev->nvqs;
>
> fail_mem:
> fail_features:
> --
> 1.8.3.1
>
>