From: Peter Stephenson <p.stephenson@samsung.com>
To: zsh-users@zsh.org
Subject: Re: How to substitute empty array element, or empty string
Date: Wed, 22 Mar 2017 09:52:12 +0000 [thread overview]
Message-ID: <20170322095212.2ff2dec8@pwslap01u.europe.root.pri> (raw)
In-Reply-To: <etPan.58d22d29.6b8b4567.10ab3@MacMini.local>
On Wed, 22 Mar 2017 08:52:09 +0100
Sebastian Gniazdowski <psprint@zdharma.org> wrote:
> Hello,
> following does not work as expected:
>
> % a=( a "" c ); print -rl -- "${(@)a//*/x}”
> x
>
> x
This might be one of those "bug" things you sometimes read about for other
projects...
I believe I've sent the version that doesn't loop infinitely.
Other cases in igetmatch() may need adaption.
pws
diff --git a/Src/glob.c b/Src/glob.c
index 0fcb4e1..9ac0ae6 100644
--- a/Src/glob.c
+++ b/Src/glob.c
@@ -2969,7 +2969,7 @@ igetmatch(char **sp, Patprog p, int fl, int n, char *replstr,
do {
/* loop over all matches for global substitution */
matched = 0;
- for (; t < send; ioff++) {
+ for (; t <= send; ioff++) {
/* Find the longest match from this position. */
set_pat_start(p, t-s);
if (pattrylen(p, t, umlen, 0, &patstralloc, ioff)) {
@@ -3018,15 +3018,19 @@ igetmatch(char **sp, Patprog p, int fl, int n, char *replstr,
* which is already marked for replacement.
*/
matched = 1;
+ if (t == send)
+ break;
while (t < mpos) {
ioff++;
umlen -= iincchar(&t, send - t);
}
break;
}
+ if (t == send)
+ break;
umlen -= iincchar(&t, send - t);
}
- } while (matched);
+ } while (matched && t < send);
/*
* check if we can match a blank string, if so do it
* at the start. Goodness knows if this is a good idea
diff --git a/Test/D04parameter.ztst b/Test/D04parameter.ztst
index cb9d50d..99f7dd9 100644
--- a/Test/D04parameter.ztst
+++ b/Test/D04parameter.ztst
@@ -2148,3 +2148,13 @@ F:behavior, see http://austingroupbugs.net/view.php?id=888
[[ ${-} = [[:alnum:]]## ]] || print Failed 2
}
0:$- expansion correctly handles Dash token
+
+ a=(1 "" 3)
+ print -rl -- "${(@)a//*/x}"
+ a=""
+ print -rl -- "${(@)a//*/y}"
+0:Zero-length string match in parameter substitution
+>x
+>x
+>x
+>y
next prev parent reply other threads:[~2017-03-23 2:03 UTC|newest]
Thread overview: 6+ messages / expand[flat|nested] mbox.gz Atom feed top
[not found] <CGME20170322075332epcas3p4a32050e48ad0f94be13134b560f17715@epcas3p4.samsung.com>
2017-03-22 7:52 ` Sebastian Gniazdowski
2017-03-22 9:52 ` Peter Stephenson [this message]
2017-03-24 11:26 ` Sebastian Gniazdowski
2017-03-24 11:49 ` Peter Stephenson
2017-03-25 6:21 ` Sebastian Gniazdowski
2017-03-25 18:59 ` Chet Ramey
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20170322095212.2ff2dec8@pwslap01u.europe.root.pri \
--to=p.stephenson@samsung.com \
--cc=zsh-users@zsh.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
Code repositories for project(s) associated with this public inbox
https://git.vuxu.org/mirror/zsh/
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).