Zsh Mailing List Archive Messages sorted by: Reverse Date, Date, Thread, Author

Re: [PATCH] Re: f() { ...; } > file

X-seq: zsh-workers 26037
From: Peter Stephenson <pws@xxxxxxx>
To: Zsh hackers list <zsh-workers@xxxxxxxxxx>
Subject: Re: [PATCH] Re: f() { ...; } > file
Date: Thu, 13 Nov 2008 14:52:59 +0000
In-reply-to: <20081113144212.GB5114@xxxxxxxxxxxxxxx>
Mailing-list: contact zsh-workers-help@xxxxxxxxxx; run by ezmlm
References: <20081105212036.GA4698@xxxxxxxxxxxxxxx> <20081113144212.GB5114@xxxxxxxxxxxxxxx>

Stephane Chazelas wrote:
> The patch below seems to fix it. It just removes the special
> case of f() { }. I don't why it was there in the first place.
> rev 1.1 of parse.c already had it.

That's because you've made all functions with braces parse as if they
contain current shell structures; you'll see they're output with an
unnecessary extra set of "{"s.  This works because it just makes the
code behave like the non-confusing way to do it, with the redirection
inside the function.

-- 
Peter Stephenson <pws@xxxxxxx>                  Software Engineer
CSR PLC, Churchill House, Cambridge Business Park, Cowley Road
Cambridge, CB4 0WZ, UK                          Tel: +44 (0)1223 692070

Follow-Ups:
- Re: [PATCH] Re: f() { ...; } > file
  - From: Stephane Chazelas

References:
- f() { ...; } > file
  - From: Stephane Chazelas
- [PATCH] Re: f() { ...; } > file
  - From: Stephane Chazelas

Messages sorted by: Reverse Date, Date, Thread, Author