Re: PATCH: another one of those

[Peter Stephenson <p.w.stephenson@xxxxxxxxxxxx>]

On Fri, 08 Oct 2010 18:04:53 +0100
Peter Stephenson <pws@xxxxxxx> wrote:
> +		    for (;;) {
> +			/*
> +			 * Not really an oddity: "\\\n" is
> +			 * removed from input as if whitespace.
> +			 */
> +			if (inblank(*pt))
> +			    pt++;
> +			else if (strpfx("\\\n", pt))
> +			    pt += 2;
> +			else
> +			    break;
> +		    }

I've just noticed we need to do this with NO_HIST_LEX_WORDS, too.
Of course it's a guess in that case, but a pretty sensible one.
Also, calling strpfx() is significant overkill for two characters in a
widely used code fragment.

Index: Src/hist.c
===================================================================
RCS file: /cvsroot/zsh/zsh/Src/hist.c,v
retrieving revision 1.105
diff -p -u -r1.105 hist.c
--- Src/hist.c	10 Oct 2010 17:26:39 -0000	1.105
+++ Src/hist.c	10 Oct 2010 17:28:52 -0000
@@ -2365,7 +2365,7 @@ readhistfile(char *fn, int err, int read
 			 */
 			if (inblank(*pt))
 			    pt++;
-			else if (strpfx("\\\n", pt))
+			else if (pt[0] == '\\' && pt[1] == '\n')
 			    pt += 2;
 			else
 			    break;
@@ -2414,8 +2414,14 @@ readhistfile(char *fn, int err, int read
 	    }
 	    if (!uselex) {
 		do {
-		    while (inblank(*pt))
-			pt++;
+		    for (;;) {
+			if (inblank(*pt))
+			    pt++;
+			else if (pt[0] == '\\' && pt[1] == '\n')
+			    pt += 2;
+			else
+			    break;
+		    }
 		    if (*pt) {
 			if (nwordpos >= nwords)
 			    words = (short *)

-- 
Peter Stephenson <p.w.stephenson@xxxxxxxxxxxx>
Web page now at http://homepage.ntlworld.com/p.w.stephenson/