qemu-trivial
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [Qemu-trivial] [Qemu-devel] [PATCH] Unnecessary comma.


From: Igor Ryzhov
Subject: Re: [Qemu-trivial] [Qemu-devel] [PATCH] Unnecessary comma.
Date: Thu, 17 Apr 2014 01:19:48 +0400
User-agent: Mozilla/5.0 (Windows NT 6.3; WOW64; rv:24.0) Gecko/20100101 Thunderbird/24.4.0

Oh, my mistake.
It's really semicolon, not comma :)

Should I resend the patch with correct name?

16.04.2014 21:58, Peter Maydell пишет:
On 16 April 2014 18:32, Stefan Weil <address@hidden> wrote:
Am 16.04.2014 15:43, schrieb Igor Ryzhov:
Signed-off-by: Igor Ryzhov <address@hidden>
---
  net/net.c | 2 +-
  1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/net/net.c b/net/net.c
index e3ef1e4..60a07f1 100644
--- a/net/net.c
+++ b/net/net.c
@@ -473,7 +473,7 @@ ssize_t qemu_deliver_packet(NetClientState *sender,

      if (ret == 0) {
          nc->receive_disabled = 1;
-    };
+    }

      return ret;
  }

Reviewed-by: Stefan Weil <address@hidden>

CC'ing qemu-trivial

PS. The "." at the end of your subject line is unnecessary, too. :-)
"net/net.c: Remove unnecessary semicolon" would probably
be better (since it lets people know which area the patch is affecting
and also gets the name of the punctuation mark right ;-)).

thanks
-- PMM




reply via email to

[Prev in Thread] Current Thread [Next in Thread]