From 07731a50fda3a25927b60c48a41be2e5b747ca01 Mon Sep 17 00:00:00 2001
From: Ralph Corderoy <ralph@inputplus.co.uk>
Date: Mon, 1 May 2017 22:57:44 +0100
Subject: [PATCH] mhlsbr.c: Flip parse()'s logic.  Remove oneline()'s else
 after break.

---
 uip/mhlsbr.c | 24 +++++++++++-------------
 1 file changed, 11 insertions(+), 13 deletions(-)

diff --git a/uip/mhlsbr.c b/uip/mhlsbr.c
index 1cc472b5..cb6080b6 100644
--- a/uip/mhlsbr.c
+++ b/uip/mhlsbr.c
@@ -848,15 +848,14 @@ parse (void)
 
     for (cp = result; *parptr && (cp - result < NAMESZ); parptr++) {
 	c = *parptr;
-	if (isalnum (c)
-		|| c == '.'
-		|| c == '-'
-		|| c == '_'
-		|| c =='['
-		|| c == ']')
-	    *cp++ = c;
-	else
+	if (!isalnum (c)
+		&& c != '.'
+		&& c != '-'
+		&& c != '_'
+		&& c !='['
+		&& c != ']')
 	    break;
+        *cp++ = c;
     }
     *cp = '\0';
 
@@ -1431,11 +1430,10 @@ oneline (char *stuff, unsigned long flags)
 		    *onelp++ = 0;
 		    break;
 		}
-		else
-		    if (!spc) {
-			*cp++ = ' ';
-			spc++;
-		    }
+                if (!spc) {
+                    *cp++ = ' ';
+                    spc++;
+                }
 	    }
 	    else {
 		*cp++ = *onelp;
-- 
2.56.0