summary refs log tree commit diff
diff options
context:
space:
mode:
authorPeter Stephenson <pws@users.sourceforge.net>2010-10-10 17:51:29 +0000
committerPeter Stephenson <pws@users.sourceforge.net>2010-10-10 17:51:29 +0000
commitaf5a85f3630f3d60ad7061fe32693baf78c27dcd (patch)
tree57a70dffb5fda8bd5cc77ea03ca7479954de9eea
parent6fea7f0d3c4d30ddc78db9e798a3ef9427abbc6f (diff)
downloadzsh-af5a85f3630f3d60ad7061fe32693baf78c27dcd.tar.gz
zsh-af5a85f3630f3d60ad7061fe32693baf78c27dcd.tar.xz
zsh-af5a85f3630f3d60ad7061fe32693baf78c27dcd.zip
28339: backslash-newline history without HIST_LEX_WORDS
28340: assignment to range of scalar variable with multibyte characters
-rw-r--r--ChangeLog10
-rw-r--r--Src/hist.c12
-rw-r--r--Src/params.c17
-rw-r--r--Test/D07multibyte.ztst18
4 files changed, 50 insertions, 7 deletions
diff --git a/ChangeLog b/ChangeLog
index a418352fd..f914bb232 100644
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,6 +1,12 @@
 2010-10-10  Peter Stephenson  <p.w.stephenson@ntlworld.com>
 
-	* 28332: Src/hist.c: HIST_LEX_WORDS didn't handled
+	* 28340: Src/params.c: assignment with negative index didn't
+	work with multibyte characters.
+
+	* 28339: Src/hist.c: NO_HIST_LEX_WORDS didn't handle
+	backslash-newline line continuation properly, either.
+
+	* 28332: Src/hist.c: HIST_LEX_WORDS didn't handle
 	backslash-newline line continuation properly.  Come to think of
 	it, neither does the alternative.
 
@@ -13728,5 +13734,5 @@
 
 *****************************************************
 * This is used by the shell to define $ZSH_PATCHLEVEL
-* $Revision: 1.5101 $
+* $Revision: 1.5102 $
 *****************************************************
diff --git a/Src/hist.c b/Src/hist.c
index 0eeb98fb0..e65ddb1b6 100644
--- a/Src/hist.c
+++ b/Src/hist.c
@@ -2365,7 +2365,7 @@ readhistfile(char *fn, int err, int readflags)
 			 */
 			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 readflags)
 	    }
 	    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 *)
diff --git a/Src/params.c b/Src/params.c
index 9a9f45893..f49a07f1a 100644
--- a/Src/params.c
+++ b/Src/params.c
@@ -2275,9 +2275,22 @@ setstrvalue(Value v, char *val)
 	    if (v->start > zlen)
 		v->start = zlen;
 	    if (v->end < 0) {
-		v->end += zlen + 1;
-		if (v->end < 0)
+		v->end += zlen;
+		if (v->end < 0) {
 		    v->end = 0;
+		} else if (v->end >= zlen) {
+		    v->end = zlen;
+		} else {
+#ifdef MULTIBYTE_SUPPORT
+		    if (isset(MULTIBYTE)) {
+			v->end += MB_METACHARLEN(z + v->end);
+		    } else {
+			v->end++;
+		    }
+#else
+		    v->end++;
+#endif
+		}
 	    }
 	    else if (v->end > zlen)
 		v->end = zlen;
diff --git a/Test/D07multibyte.ztst b/Test/D07multibyte.ztst
index 53cbe2a1d..2cb995346 100644
--- a/Test/D07multibyte.ztst
+++ b/Test/D07multibyte.ztst
@@ -447,3 +447,21 @@
   print $(( [#16] #REPLY ))
 0:read passes through invalid multibyte characters
 >0xC5
+
+  word=abcま          
+  word[-1]=
+  print $word
+  word=abcま 
+  word[-2]=
+  print $word
+  word=abcま 
+  word[4]=d
+  print $word
+  word=abcま 
+  word[3]=not_c
+  print $word  
+0:assignment with negative indices
+>abc
+>abま
+>abcd
+>abnot_cま