From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 27543 invoked by alias); 5 Sep 2013 14:26:11 -0000 Mailing-List: contact zsh-workers-help@zsh.org; run by ezmlm Precedence: bulk X-No-Archive: yes List-Id: Zsh Workers List List-Post: List-Help: X-Seq: 31700 Received: (qmail 28187 invoked from network); 5 Sep 2013 14:25:55 -0000 X-Spam-Checker-Version: SpamAssassin 3.3.2 (2011-06-06) on f.primenet.com.au X-Spam-Level: X-Spam-Status: No, score=-1.9 required=5.0 tests=BAYES_00,RCVD_IN_DNSWL_NONE autolearn=ham version=3.3.2 Received-SPF: none (ns1.primenet.com.au: domain at closedmail.com does not designate permitted sender hosts) From: Bart Schaefer Message-id: <130905072545.ZM29595@torch.brasslantern.com> Date: Thu, 05 Sep 2013 07:25:45 -0700 In-reply-to: <20130904200929.5028f433@pws-pc.ntlworld.com> Comments: In reply to Peter Stephenson "Re: Builtin test and parsing of conditionals" (Sep 4, 8:09pm) References: <130904091503.ZM28369@torch.brasslantern.com> <20130904173159.19c55cd1@pwslap01u.europe.root.pri> <130904103940.ZM28454@torch.brasslantern.com> <20130904200929.5028f433@pws-pc.ntlworld.com> X-Mailer: OpenZMail Classic (0.9.2 24April2005) To: zsh-workers@zsh.org Subject: Re: Builtin test and parsing of conditionals MIME-version: 1.0 Content-type: text/plain; charset=us-ascii On Sep 4, 8:09pm, Peter Stephenson wrote: } } On Wed, 04 Sep 2013 10:39:40 -0700 } Bart Schaefer wrote: } > Sorry, perhaps I should have been clearer: "! -a !" means to compare the } > truth value "!" to the truth value of "!". } } OK, this looks straightforward enough. That's good, but it still doesn't fix the other case mentioned in my original email: torch% /usr/bin/test ! = -a o ; print $? 1 torch% test ! = -a o ; print $? test: too many arguments 1 Do you want to try to handle that in parse.c, or am I correct that peeling off the "!" in builtin.c:bin_test [the way parens are peeled off] is the better way to go? Either way, the comment in builtin.c should be updated with the new URL path to test.html that Chet was kind enough to provide. The one that's there now is from 2007.