asm/preproc.c: remove dead code, improve comment

ppscan() had a break; statement followed immediately by a return
statement. The latter was left over from code restructuring and is
dead code.

Remove it.

Fix a nearby comment indicating that we should not need to strip a
const from a string pointer. Probably Token::t_charptr should be
const, but it might need additional work.

Signed-off-by: H. Peter Anvin (Intel) <hpa@zytor.com>
diff --git a/asm/preproc.c b/asm/preproc.c
index 75e4aac..85606e0 100644
--- a/asm/preproc.c
+++ b/asm/preproc.c
@@ -2283,12 +2283,11 @@
     txt = tok_text(tline);
     tokval->t_start = txt;
     tokval->t_len = tline->len;
-    tokval->t_charptr = (char *)txt; /* Fix this */
+    tokval->t_charptr = (char *)txt; /* Fix needing const removal here */
 
     switch (tline->type) {
     default:
         break;
-        return tokval->t_type = tline->type;
 
     case TOKEN_ID:
         if (txt[0] == '$') {