This is an automated email from Gerrit.

Antonio Borneo (borneo.anto...@gmail.com) just uploaded a new patch set to 
Gerrit, which you can find at http://openocd.zylin.com/87

-- gerrit

commit 31213ac4e572e0f515ab0fee542d4054d5de5038
Author: Antonio Borneo <borneo.anto...@gmail.com>
Date:   Sun Oct 23 12:16:26 2011 +0800

    SERVER: fix clang warning
    
    The fix is inline with the Linux coding style that forbids
    assignment in if condition
    
    Change-Id: I0b9d0b419d9c8b7a8c755e048d5faf72d1658ba2
    Signed-off-by: Antonio Borneo <borneo.anto...@gmail.com>

diff --git a/src/server/server.c b/src/server/server.c
index bb60fc5..a7fddf6 100644
--- a/src/server/server.c
+++ b/src/server/server.c
@@ -487,7 +487,8 @@ int server_loop(struct command_context *command_context)
                                {
                                        if ((FD_ISSET(c->fd, &read_fds)) || 
c->input_pending)
                                        {
-                                               if ((retval = 
service->input(c)) != ERROR_OK)
+                                               retval = service->input(c);
+                                               if (retval != ERROR_OK)
                                                {
                                                        struct connection *next 
= c->next;
                                                        if (service->type == 
CONNECTION_PIPE)

-- 
_______________________________________________
Openocd-development mailing list
Openocd-development@lists.berlios.de
https://lists.berlios.de/mailman/listinfo/openocd-development

Reply via email to