From: Peter Stephenson <p.w.stephenson@ntlworld.com>
To: zsh-workers@zsh.org
Subject: Re: (LC_ALL=C; set -x 128 129; printf "%s\n" ${(#)@} | hexdump -C)
Date: Thu, 7 Sep 2023 17:33:53 +0100 (BST) [thread overview]
Message-ID: <899459233.232418.1694104433053@mail.virginmedia.com> (raw)
In-Reply-To: <EBF09792-A011-44F6-A36C-8F704A1A1FF8@kba.biglobe.ne.jp>
> On 07/09/2023 15:26 Jun. T <takimoto-j@kba.biglobe.ne.jp> wrote:
> > 2023/09/01 2:38, I wrote:
> > So we need to reset (or save/restore) errflag somewhere...
>
> The patch below is a simple save/restore errflag. It seems to work,
> but I'm not sure it is the (or a) correct fix. I will not push this
> unless I get positive responses.
Looks like these are the only calls to substevalchar(), so perhaps
the changes could be made internal to that. If the test for
errflag within substevalchar() is, as I presume, there simply because
something later in the function will fail if errflag is non-zero, that
might be the point to clear it. Then the function can return the
error state at the end of the function just before it restores errflag.
pws
next prev parent reply other threads:[~2023-09-07 16:34 UTC|newest]
Thread overview: 10+ messages / expand[flat|nested] mbox.gz Atom feed top
2023-08-30 7:27 Stephane Chazelas
2023-08-30 19:45 ` Bart Schaefer
2023-08-31 17:38 ` Jun. T
2023-09-07 14:26 ` Jun. T
2023-09-07 16:33 ` Peter Stephenson [this message]
2023-09-08 16:30 ` Jun. T
2023-09-11 8:57 ` Peter Stephenson
2023-09-11 12:11 ` Jun. T
2023-09-13 9:59 ` Jun T
2023-09-21 9:26 ` Jun T
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=899459233.232418.1694104433053@mail.virginmedia.com \
--to=p.w.stephenson@ntlworld.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
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).