]> asedeno.scripts.mit.edu Git - PuTTY.git/commitdiff
Avoid -Wmisleading-indentation warnings with GCC 6.
authorColin Watson <cjwatson@debian.org>
Thu, 28 Jan 2016 21:19:41 +0000 (21:19 +0000)
committerSimon Tatham <anakin@pobox.com>
Sat, 30 Jan 2016 14:33:54 +0000 (14:33 +0000)
GCC 6 warns about potentially misleading indentation, such as:

    if (condition) stmt1; stmt2;

Chaining multiple ifs on a single line runs into this warning, even if
it's probably not actually misleading to a human eye, so just add a
couple of newlines to pacify the compiler.

cproxy.c
proxy.c

index 5f2f15fc73ca687b5f65f92476a1a6b0f60c72b3..4f9a3ffa591c8b00833d0f6b24c69f83ca64ec06 100644 (file)
--- a/cproxy.c
+++ b/cproxy.c
@@ -173,7 +173,8 @@ int proxy_socks5_selectchap(Proxy_Socket p)
        chapbuf[5] = '\x02'; /* Second attribute - username */
 
        ulen = strlen(username);
-       if (ulen > 255) ulen = 255; if (ulen < 1) ulen = 1;
+       if (ulen > 255) ulen = 255;
+       if (ulen < 1) ulen = 1;
 
        chapbuf[6] = ulen;
        memcpy(chapbuf+7, username, ulen);
diff --git a/proxy.c b/proxy.c
index 5b77bc7670b1844a273c1693042502ebf39672e6..f9f2cd62df8de99f521dc9588c89be35b5b22f38 100644 (file)
--- a/proxy.c
+++ b/proxy.c
@@ -1246,9 +1246,11 @@ int proxy_socks5_negotiate (Proxy_Socket p, int change)
                char userpwbuf[255 + 255 + 3];
                int ulen, plen;
                ulen = strlen(username);
-               if (ulen > 255) ulen = 255; if (ulen < 1) ulen = 1;
+               if (ulen > 255) ulen = 255;
+               if (ulen < 1) ulen = 1;
                plen = strlen(password);
-               if (plen > 255) plen = 255; if (plen < 1) plen = 1;
+               if (plen > 255) plen = 255;
+               if (plen < 1) plen = 1;
                userpwbuf[0] = 1;      /* version number of subnegotiation */
                userpwbuf[1] = ulen;
                memcpy(userpwbuf+2, username, ulen);