[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-block] [RFC v1 2/4] util/oslib-win32: Remove invalid check
From: |
Fam Zheng |
Subject: |
Re: [Qemu-block] [RFC v1 2/4] util/oslib-win32: Remove invalid check |
Date: |
Thu, 29 Jun 2017 17:25:34 +0800 |
User-agent: |
Mutt/1.8.0 (2017-02-23) |
On Tue, 06/27 16:57, Alistair Francis wrote:
> There is no way nhandles can be zero in this section so that part of the
> if statement will always be false. Let's just remove it to make the code
> easier to read.
>
> Signed-off-by: Alistair Francis <address@hidden>
> Acked-by: Edgar E. Iglesias <address@hidden>
> ---
>
> util/oslib-win32.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/util/oslib-win32.c b/util/oslib-win32.c
> index 80e4668935..7ec0f8e083 100644
> --- a/util/oslib-win32.c
> +++ b/util/oslib-win32.c
> @@ -414,7 +414,7 @@ static int poll_rest(gboolean poll_msgs, HANDLE *handles,
> gint nhandles,
> /* If we have a timeout, or no handles to poll, be satisfied
> * with just noticing we have messages waiting.
> */
> - if (timeout != 0 || nhandles == 0) {
> + if (timeout != 0) {
> return 1;
> }
>
> --
> 2.11.0
>
Reviewed-by: Fam Zheng <address@hidden>
[Qemu-block] [RFC v1 1/4] util/aio-win32: Only select on what we are actually waiting for, Alistair Francis, 2017/06/27