zsh-workers
 help / color / Atom feed
From: Sebastian Gniazdowski <sgniazdowski@gmail.com>
To: Zsh hackers list <zsh-workers@zsh.org>
Subject: [BUG] Appending by arr[@]//(#e)/... crashes Zsh on WSL
Date: Mon, 21 Oct 2019 15:30:05 +0200
Message-ID: <CAKc7PVAKrRviCrvvXfkmyHFBz3O4urMZqiPEc-gt-L9mVTJpoA@mail.gmail.com> (raw)

Hello,
I've just fixed a long standing bug in F-Sy-H:

https://github.com/zdharma/fast-syntax-highlighting/issues/146

by a patch that changed appending to an array from:

arr=( ${arr[@]//(#e)/an-appendage} )

with a manual loop:

for __ in ${arr[@]}; do
__s+=( ${__}an-appendage )
done

The exact commit is:
https://github.com/zdharma/fast-syntax-highlighting/commit/9a6f

I guess that a Valgrind run would be helpful. BTW. How are the
automatic Valgrind tests doing? They would be helpful now.

-- 
Sebastian Gniazdowski
News: https://twitter.com/ZdharmaI
IRC: https://kiwiirc.com/client/chat.freenode.net:+6697/#zplugin
Blog: http://zdharma.org

             reply index

Thread overview: 2+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2019-10-21 13:30 Sebastian Gniazdowski [this message]
2019-10-21 13:35 ` Sebastian Gniazdowski

Reply instructions:

You may reply publically 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=CAKc7PVAKrRviCrvvXfkmyHFBz3O4urMZqiPEc-gt-L9mVTJpoA@mail.gmail.com \
    --to=sgniazdowski@gmail.com \
    --cc=zsh-workers@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

zsh-workers

Archives are clonable: git clone --mirror http://inbox.vuxu.org/zsh-workers

Example config snippet for mirrors

Newsgroup available over NNTP:
	nntp://inbox.vuxu.org/vuxu.archive.zsh.workers


AGPL code for this site: git clone https://public-inbox.org/public-inbox.git