zsh-workers
 help / color / mirror / code / Atom feed
From: Sven Wischnowsky <wischnow@informatik.hu-berlin.de>
To: zsh-workers@sunsite.auc.dk
Subject: Re: PATCH: 3.1.6-dev-18 AIX dependency fixes
Date: Mon, 14 Feb 2000 10:44:50 +0100 (MET)	[thread overview]
Message-ID: <200002140944.KAA09412@beta.informatik.hu-berlin.de> (raw)
In-Reply-To: Peter Stephenson's message of Sun, 13 Feb 2000 18:43:06 +0000


Peter Stephenson wrote:

> ...
>
> There are other things --- for example, I suspect getting and setting of
> parameters internally needs a complete rethink --- but it's unrealistic to
> put a timetable on them.

Having done quite a bit of testing lately, I'd say: yes. It's
currently the slowest part of the shell, and used quite often.

> Since completion now appears to be pretty much stable, 4.0.1 could then
> appear without too many user-visible changes.  However, I think the
> wordcode stuff is going to continue to evolve anyway (rewrite parser to
> eliminate the old-style structs altogether, aim to eliminate
> HEAPALLOC/PERMALLOC by passing allocation strategy directly to appropriate
> constructors), and I don't think it would be a good idea to produce 4.0.1
> with the current hybrid state, though it won't matter for 3.1.7.

Well, I've written: a) a parser that immediatly produces the wordcode
(still recursive), b) a non-recursive execution code c) several
optimisations.

Unfortunately, b) doesn't seem to make things faster and probably less 
easy to read (I've only tested this on a Intel/Linux box yet, it may
be different on other machines). And, unfortunately, I overwrote the
patch for a) alone, so now I have it only for a) and c) together. I'll 
try to separate them again, but I don't know when I'll find the time.

And none of these needs to be in 3.1.7, of course.

Bye
 Sven

P.S.: c) is quite good, almost 30% faster than before.

--
Sven Wischnowsky                         wischnow@informatik.hu-berlin.de


             reply	other threads:[~2000-02-14  9:45 UTC|newest]

Thread overview: 3+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2000-02-14  9:44 Sven Wischnowsky [this message]
  -- strict thread matches above, loose matches on Subject: below --
2000-02-12 16:47 Oliver Kiddle
2000-02-13 18:43 ` 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=200002140944.KAA09412@beta.informatik.hu-berlin.de \
    --to=wischnow@informatik.hu-berlin.de \
    --cc=zsh-workers@sunsite.auc.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).