qemu-devel.nongnu.org archive mirror
 help / color / mirror / Atom feed
* [Qemu-devel] [PATCH] vhost: remove unnecessary code
@ 2015-08-20  4:28 Yuanhan Liu
  2015-08-21 11:32 ` Marc-André Lureau
  0 siblings, 1 reply; 2+ messages in thread
From: Yuanhan Liu @ 2015-08-20  4:28 UTC (permalink / raw)
  To: qemu-devel; +Cc: Yuanhan Liu, Michael S. Tsirkin

We won't reference var 'i' anymore; there is no need to assign a value
to it.

cc: Michael S. Tsirkin <mst@redhat.com>
Signed-off-by: Yuanhan Liu <yuanhan.liu@linux.intel.com>
---
 hw/virtio/vhost.c | 1 -
 1 file changed, 1 deletion(-)

diff --git a/hw/virtio/vhost.c b/hw/virtio/vhost.c
index 2712c6f..86ea26f 100644
--- a/hw/virtio/vhost.c
+++ b/hw/virtio/vhost.c
@@ -1158,7 +1158,6 @@ fail_vq:
                              hdev->vqs + i,
                              hdev->vq_index + i);
     }
-    i = hdev->nvqs;
 fail_mem:
 fail_features:
 
-- 
1.9.0

^ permalink raw reply related	[flat|nested] 2+ messages in thread

* Re: [Qemu-devel] [PATCH] vhost: remove unnecessary code
  2015-08-20  4:28 [Qemu-devel] [PATCH] vhost: remove unnecessary code Yuanhan Liu
@ 2015-08-21 11:32 ` Marc-André Lureau
  0 siblings, 0 replies; 2+ messages in thread
From: Marc-André Lureau @ 2015-08-21 11:32 UTC (permalink / raw)
  To: Yuanhan Liu; +Cc: QEMU, Michael S. Tsirkin

Hi

On Thu, Aug 20, 2015 at 6:28 AM, Yuanhan Liu
<yuanhan.liu@linux.intel.com> wrote:
> We won't reference var 'i' anymore; there is no need to assign a value
> to it.
>
> cc: Michael S. Tsirkin <mst@redhat.com>
> Signed-off-by: Yuanhan Liu <yuanhan.liu@linux.intel.com>
> ---
>  hw/virtio/vhost.c | 1 -
>  1 file changed, 1 deletion(-)
>
> diff --git a/hw/virtio/vhost.c b/hw/virtio/vhost.c
> index 2712c6f..86ea26f 100644
> --- a/hw/virtio/vhost.c
> +++ b/hw/virtio/vhost.c
> @@ -1158,7 +1158,6 @@ fail_vq:
>                               hdev->vqs + i,
>                               hdev->vq_index + i);
>      }
> -    i = hdev->nvqs;
>  fail_mem:
>  fail_features:
>

Reviewed-by: Marc-André Lureau <marcandre.lureau@redhat.com>






-- 
Marc-André Lureau

^ permalink raw reply	[flat|nested] 2+ messages in thread

end of thread, other threads:[~2015-08-21 11:32 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2015-08-20  4:28 [Qemu-devel] [PATCH] vhost: remove unnecessary code Yuanhan Liu
2015-08-21 11:32 ` Marc-André Lureau

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).