Zsh Mailing List Archive
Messages sorted by:
Reverse Date,
Date,
Thread,
Author
Re: Bug: cd auto-completion of .. fails with parentheses in directory name
- X-seq: zsh-workers 40021
- From: Daniel Shahaf <d.s@xxxxxxxxxxxxxxxxxx>
- To: Mikael Magnusson <mikachu@xxxxxxxxx>
- Subject: Re: Bug: cd auto-completion of .. fails with parentheses in directory name
- Date: Sun, 27 Nov 2016 15:17:06 +0000
- Cc: zsh workers <zsh-workers@xxxxxxx>
- Dkim-signature: v=1; a=rsa-sha1; c=relaxed/relaxed; d= daniel.shahaf.name; h=cc:content-type:date:from:in-reply-to :message-id:mime-version:references:subject:to:x-me-sender :x-me-sender:x-sasl-enc:x-sasl-enc; s=mesmtp; bh=VA1VMJD7CMiMHRS J+Zw1ZCl9Ig8=; b=nRD08O+Eal6hui9Rzu4ncxSqb/prz4RTR03bsu1qt3qr0/7 EeeyOBidNtxCIg/dAgzADr0t8TCly1/PzB03sFtjt9+hBk+7iZV6u1JG3gjXA7kO 3Nl/AkMploB3NoXLeMUcDOxxo29vVoIIhe9Uaz33n4lPV4ejirP32oTJqCtc=
- Dkim-signature: v=1; a=rsa-sha1; c=relaxed/relaxed; d= messagingengine.com; h=cc:content-type:date:from:in-reply-to :message-id:mime-version:references:subject:to:x-me-sender :x-me-sender:x-sasl-enc:x-sasl-enc; s=smtpout; bh=VA1VMJD7CMiMHR SJ+Zw1ZCl9Ig8=; b=Tsi+yhG6GyThD1CFN3oIyoo1MSESMnSHtzt4jwNOwAnLaW 2V0XbOy1vJx7KwmsJup/+Xr6UTCv9qqD/WGIpENSuKFaz1dsRhMhfKhAC+AJkfDS ZrePF049Ni2znCMu+P0+lBBikMWbvji44E2PQXCwFMleXBLT8TqKx8NaQJZNU=
- In-reply-to: <20161029180632.GA18192@fujitsu.shahaf.local2>
- List-help: <mailto:zsh-workers-help@zsh.org>
- List-id: Zsh Workers List <zsh-workers.zsh.org>
- List-post: <mailto:zsh-workers@zsh.org>
- Mailing-list: contact zsh-workers-help@xxxxxxx; run by ezmlm
- References: <CAPnWG_TR7DC7h36m-jp6PnuY=M4ViwwWNMtaT9aU4wawygzoBg@mail.gmail.com> <20160922144250.GA11076@fujitsu.shahaf.local2> <160922103042.ZM7407@torch.brasslantern.com> <CAHYJk3QmMUr0_C0-aAMYCNsmeXuo14JzVXwuimBk05meeX88wA@mail.gmail.com> <20161028151037.GA10483@fujitsu.shahaf.local2> <20161029180632.GA18192@fujitsu.shahaf.local2>
Daniel Shahaf wrote on Sat, Oct 29, 2016 at 18:06:32 +0000:
> diff --git a/Completion/Unix/Type/_path_files b/Completion/Unix/Type/_path_files
> index 0d36b54..32942d7 100644
> --- a/Completion/Unix/Type/_path_files
> +++ b/Completion/Unix/Type/_path_files
> @@ -594,7 +594,7 @@ for prepath in "$prepaths[@]"; do
> # There are more components, so skip over the next components and make a
> # slash be added.
>
> - tmp1=( ${tmp1//(#b)([][()|*?^#~<>\\=])/\\${match[1]}} )
> + #tmp1=( ${tmp1//(#b)([][()|*?^#~<>\\=])/\\${match[1]}} )
> tmp2="${(M)tpre##${~skips}}"
> if [[ -n "$tmp2" ]]; then
> skipped="/$tmp2"
I've reviewed the code again. I believe that at the preceding
'compfiles -i' call, tmp1 is an input parameter containing a list of
unquoted filenames, and is not modified by the call, so this patch is
correct.
Committed in eccb7471b577d55f0b410088fc1125016476b332, with a regression
test.
I'd add some reverse-engineered docs for bin_compfiles()'s -i case but
I can't type right now.
@all, please report breakage if any, with completiosn of oddly-named
directories.
Cheers,
Daniel
Messages sorted by:
Reverse Date,
Date,
Thread,
Author