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

Re: [wip patch] new zsh/attr module

X-seq: zsh-workers 26673
From: Peter Stephenson <pws@xxxxxxx>
To: zsh workers <zsh-workers@xxxxxxxxxx>
Subject: Re: [wip patch] new zsh/attr module
Date: Tue, 03 Mar 2009 16:55:08 +0000
In-reply-to: <237967ef0903030851gc26620ficfc908628a4b3be2@xxxxxxxxxxxxxx>
Mailing-list: contact zsh-workers-help@xxxxxxxxxx; run by ezmlm
References: <alpine.LNX.2.00.0902262245590.27571@localhost> <20090303121253.61f5e2ec@news01> <alpine.LNX.2.00.0903031455360.10365@localhost> <20090303163526.533995be@news01> <237967ef0903030851gc26620ficfc908628a4b3be2@xxxxxxxxxxxxxx>

Mikael Magnusson wrote:
> > don't report errors unless system calls returned a negative value
> 
> I know the codestyle is a bit questionable but I already check 0 < len
> in the first if, so the else will be all negative lengths already.

... or zero, which was the problem.  It was printing an error message
with errno 0 if the list returned empty.  It now does the standard
Unix-style thing of printing nothing but returning status zero.

-- 
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: [wip patch] new zsh/attr module
  - From: Mikael Magnusson

References:
- [wip patch] new zsh/attr module
  - From: Mikael Magnusson
- Re: [wip patch] new zsh/attr module
  - From: Peter Stephenson
- Re: [wip patch] new zsh/attr module
  - From: Mikael Magnusson
- Re: [wip patch] new zsh/attr module
  - From: Peter Stephenson
- Re: [wip patch] new zsh/attr module
  - From: Mikael Magnusson

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