From: Mikael Magnusson <mikachu@gmail.com>
To: Peter Stephenson <Peter.Stephenson@csr.com>
Cc: "Zsh Hackers' List" <zsh-workers@zsh.org>
Subject: Re: MAX_ARRLEN
Date: Mon, 23 Apr 2012 18:36:22 +0200 [thread overview]
Message-ID: <CAHYJk3T5x+tFan+CSLH6srijXiET_PxrxoTjDEsL5CUieMq+hw@mail.gmail.com> (raw)
In-Reply-To: <20120423172723.238137e1@pwslap01u.europe.root.pri>
On 2012-04-23, Peter Stephenson <Peter.Stephenson@csr.com> wrote:
> On Mon, 23 Apr 2012 09:21:23 -0700
> Bart Schaefer <schaefer@brasslantern.com> wrote:
>> On Apr 23, 6:10pm, Mikael Magnusson wrote:
>> }
>> } http://www.zsh.org/mla/workers/2010/msg00013.html
>>
>> And
>>
>> http://www.zsh.org/mla/workers/2010/msg00015.html
>
> Those are basically saying yes, the current set up has problems but we'd
> quite like something. Hence my immediate suggestions of what we
> *actually* do.
>
> The only additional matter arising is that it appears quite a lot of
> people would be happy with the limit defaulting off.
I replied with the link because you didn't refer to the previous
discussion at all, so I wasn't sure if you remembered it :).
The problem with the current approach is that it only limits accessing
an array beyond a certain index, even if it's already larger, and you
can create arrays of any size by other means. To me it seems like the
limit is applied in the wrong place at least. If there are places that
unexpectedly create large arrays, we should add the safeguards in
those places, if possible.
--
Mikael Magnusson
next prev parent reply other threads:[~2012-04-23 16:36 UTC|newest]
Thread overview: 11+ messages / expand[flat|nested] mbox.gz Atom feed top
2012-04-23 15:27 MAX_ARRLEN Peter Stephenson
2012-04-23 16:10 ` MAX_ARRLEN Mikael Magnusson
2012-04-23 16:21 ` MAX_ARRLEN Bart Schaefer
2012-04-23 16:27 ` MAX_ARRLEN Peter Stephenson
2012-04-23 16:36 ` Mikael Magnusson [this message]
2012-04-23 16:40 ` MAX_ARRLEN Peter Stephenson
2012-04-23 16:45 ` MAX_ARRLEN Mikael Magnusson
2012-04-23 16:38 ` MAX_ARRLEN Bart Schaefer
2012-04-24 13:37 ` MAX_ARRLEN Peter Stephenson
2012-04-24 19:45 ` MAX_ARRLEN Bart Schaefer
2012-04-25 9:01 ` MAX_ARRLEN 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=CAHYJk3T5x+tFan+CSLH6srijXiET_PxrxoTjDEsL5CUieMq+hw@mail.gmail.com \
--to=mikachu@gmail.com \
--cc=Peter.Stephenson@csr.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).