From: "Bart Schaefer" <schaefer@brasslantern.com>
To: Peter Stephenson <pws@csr.com>,
zsh-workers@sunsite.dk (Zsh hackers list)
Subject: PATCH Re: Parameter error trapped too late
Date: Mon, 16 Sep 2002 15:00:17 +0000 [thread overview]
Message-ID: <1020916150017.ZM7997@candle.brasslantern.com> (raw)
In-Reply-To: <1444.1032185019@csr.com>
On Sep 16, 3:03pm, Peter Stephenson wrote:
} Subject: Parameter error trapped too late
}
} % zsh -c 'fn() { local 30=foo; }; fn'
} BUG: parameter recreated with wrong flags
} fn:local: 30: can't assign initial value for array
The simplest fix appears to be the following patch, which has the effect
that instead of this error:
schaefer<501> typeset 30=foo
zsh: 30: can't assign initial value for array
One gets this error:
schaefer<501> typeset 30=foo
zsh: not an identifier: 30
We could refine the error message if that's desirable, but bash2 gives a
similar "not a valid identifier" in that example.
Index: builtin.c
===================================================================
diff -c -r1.17 builtin.c
--- builtin.c 12 Sep 2002 07:59:07 -0000 1.17
+++ builtin.c 16 Sep 2002 14:49:34 -0000
@@ -1909,7 +1909,7 @@
"%s: array elements must be scalar", pname, 0);
return NULL;
}
- } else if (isident(pname)) {
+ } else if (isident(pname) && !idigit(*pname)) {
/*
* Create a new node for a parameter with the flags in `on' minus the
* readonly flag
Index: B02typeset.ztst
===================================================================
diff -c -r1.2 B02typeset.ztst
--- B02typeset.ztst 1 Sep 2002 16:47:40 -0000 1.2
+++ B02typeset.ztst 16 Sep 2002 14:49:38 -0000
@@ -223,7 +223,7 @@
>scalar a r y
# The first declare works around the "not an identifier" bug with -h
- declare \! \# \$ * - ? @
+ declare \! \# \$ * - ? @ 0
typeset -h +g -m *
unset -m *
integer i=9
--
Bart Schaefer Brass Lantern Enterprises
http://www.well.com/user/barts http://www.brasslantern.com
Zsh: http://www.zsh.org | PHPerl Project: http://phperl.sourceforge.net
next prev parent reply other threads:[~2002-09-16 15:00 UTC|newest]
Thread overview: 5+ messages / expand[flat|nested] mbox.gz Atom feed top
2002-09-16 14:03 Peter Stephenson
2002-09-16 15:00 ` Bart Schaefer [this message]
2002-09-16 15:59 ` PATCH " Peter Stephenson
2002-09-18 15:07 ` PATCH (revised) " Bart Schaefer
2002-09-18 17:47 ` Peter Stephenson
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=1020916150017.ZM7997@candle.brasslantern.com \
--to=schaefer@brasslantern.com \
--cc=pws@csr.com \
--cc=zsh-workers@sunsite.dk \
/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).