about summary refs log tree commit diff
path: root/Src/subst.c
diff options
context:
space:
mode:
authorBarton E. Schaefer <schaefer@zsh.org>2016-08-28 19:12:17 -0700
committerBarton E. Schaefer <schaefer@zsh.org>2016-08-29 08:34:03 -0700
commitfa48711e31c4d45819455e767ad106c41f9a92fe (patch)
tree422ed5d5bcaca3c522e436b2629292013fc3cfb3 /Src/subst.c
parent44a820f1c5dce90d07f6cf2e2af3348bee52fa13 (diff)
downloadzsh-fa48711e31c4d45819455e767ad106c41f9a92fe.tar.gz
zsh-fa48711e31c4d45819455e767ad106c41f9a92fe.tar.xz
zsh-fa48711e31c4d45819455e767ad106c41f9a92fe.zip
39115: repair forced joining when (@) and (j) are used together
Diffstat (limited to 'Src/subst.c')
-rw-r--r--Src/subst.c5
1 files changed, 3 insertions, 2 deletions
diff --git a/Src/subst.c b/Src/subst.c
index 15eb59b64..4641b4ba5 100644
--- a/Src/subst.c
+++ b/Src/subst.c
@@ -3458,7 +3458,8 @@ paramsubst(LinkList l, LinkNode n, char **str, int qt, int pf_flags,
     if (ssub || spbreak || spsep || sep) {
 	int force_split = !ssub && (spbreak || spsep);
 	if (isarr) {
-	    if (nojoin == 0) {
+	    /* sep non-null here means F or j flag, force join */
+	    if (nojoin == 0 || sep) {
 		val = sepjoin(aval, sep, 1);
 		isarr = 0;
 		ms_flags = 0;
@@ -3467,7 +3468,7 @@ paramsubst(LinkList l, LinkNode n, char **str, int qt, int pf_flags,
 		 * forced joining as previously determined, or
 		 * join on what we later use to forcibly split
 		 */
-		val = sepjoin(aval, (nojoin == 1 ? sep : spsep), 1);
+		val = sepjoin(aval, (nojoin == 1 ? NULL : spsep), 1);
 		isarr = 0;
 	    }
 	}