[PATCH net-next 6/7] netconsole: improve code style in parser function

[Breno Leitao]

Split assignment from conditional checks and use preferred null pointer
check style (!delim instead of == NULL) in netconsole_parser_cmdline().
This improves code readability and follows kernel coding style
conventions.

Signed-off-by: Breno Leitao <lei...@debian.org>
---
 drivers/net/netconsole.c | 15 ++++++++++-----
 1 file changed, 10 insertions(+), 5 deletions(-)

diff --git a/drivers/net/netconsole.c b/drivers/net/netconsole.c
index 3406a5cbdc11a..2a13b6f16f082 100644
--- a/drivers/net/netconsole.c
+++ b/drivers/net/netconsole.c
@@ -1659,7 +1659,8 @@ static int netconsole_parser_cmdline(struct netpoll *np, 
char *opt)
        int ipv6;
 
        if (*cur != '@') {
-               if ((delim = strchr(cur, '@')) == NULL)
+               delim = strchr(cur, '@');
+               if (!delim)
                        goto parse_failed;
                *delim = 0;
                if (kstrtou16(cur, 10, &np->local_port))
@@ -1670,7 +1671,8 @@ static int netconsole_parser_cmdline(struct netpoll *np, 
char *opt)
 
        if (*cur != '/') {
                ipversion_set = true;
-               if ((delim = strchr(cur, '/')) == NULL)
+               delim = strchr(cur, '/');
+               if (!delim)
                        goto parse_failed;
                *delim = 0;
                ipv6 = netpoll_parse_ip_addr(cur, &np->local_ip);
@@ -1684,7 +1686,8 @@ static int netconsole_parser_cmdline(struct netpoll *np, 
char *opt)
 
        if (*cur != ',') {
                /* parse out dev_name or dev_mac */
-               if ((delim = strchr(cur, ',')) == NULL)
+               delim = strchr(cur, ',');
+               if (!delim)
                        goto parse_failed;
                *delim = 0;
 
@@ -1701,7 +1704,8 @@ static int netconsole_parser_cmdline(struct netpoll *np, 
char *opt)
 
        if (*cur != '@') {
                /* dst port */
-               if ((delim = strchr(cur, '@')) == NULL)
+               delim = strchr(cur, '@');
+               if (!delim)
                        goto parse_failed;
                *delim = 0;
                if (*cur == ' ' || *cur == '\t')
@@ -1713,7 +1717,8 @@ static int netconsole_parser_cmdline(struct netpoll *np, 
char *opt)
        cur++;
 
        /* dst ip */
-       if ((delim = strchr(cur, '/')) == NULL)
+       delim = strchr(cur, '/');
+       if (!delim)
                goto parse_failed;
        *delim = 0;
        ipv6 = netpoll_parse_ip_addr(cur, &np->remote_ip);

-- 
2.47.1