From mboxrd@z Thu Jan  1 00:00:00 1970
Return-Path: <zsh-workers-return-29564-mason-zsh=primenet.com.au@zsh.org>
Received: (qmail 17873 invoked by alias); 20 Jul 2011 11:54:06 -0000
Mailing-List: contact zsh-workers-help@zsh.org; run by ezmlm
Precedence: bulk
X-No-Archive: yes
List-Id: Zsh Workers List <zsh-workers.zsh.org>
List-Post: <mailto:zsh-workers@zsh.org>
List-Help: <mailto:zsh-workers-help@zsh.org>
X-Seq: 29564
Received: (qmail 1905 invoked from network); 20 Jul 2011 11:54:03 -0000
X-Spam-Checker-Version: SpamAssassin 3.3.1 (2010-03-16) on f.primenet.com.au
X-Spam-Level: 
X-Spam-Status: No, score=-2.6 required=5.0 tests=BAYES_00,DKIM_SIGNED,
	DKIM_VALID,RCVD_IN_DNSWL_LOW autolearn=ham version=3.3.1
Received-SPF: pass (ns1.primenet.com.au: SPF record at _spf.google.com designates 74.125.83.171 as permitted sender)
DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed;
        d=gmail.com; s=gamma;
        h=mime-version:sender:in-reply-to:references:date
         :x-google-sender-auth:message-id:subject:from:to:cc:content-type
         :content-transfer-encoding;
        bh=T/ekRae+kFT7oeeRyRmth4A3LbISz90MZHo3YbN4rao=;
        b=Dwnf2ElPZJ6SuS3rn0Z4/c1hOi8Tmk1HCs9CkonFGihtGuzFcMkGpp94tsb/+WdAeU
         bwmlqkCb3OTFOcBC5FKeBLQ+EM61eI9bC24+bqlzoDDLTRG/lqa9Xwt2YlWz2N+y3Ydn
         1t1VlQpg4yPpYEBCwLhYoeTIoz7v5GD3oNd44=
MIME-Version: 1.0
Sender: nikolai.weibull@gmail.com
In-Reply-To: <BANLkTi=pMjD5rBJRaZrj5FLWVbndGAMGYg@mail.gmail.com>
References: <BANLkTinV2euaa_w+F7m7U0KMZm1ajYDMUg@mail.gmail.com>
	<BANLkTikbLN=MTgHsiQeNsbwYXMaJ7GVP+w@mail.gmail.com>
	<BANLkTi=pMjD5rBJRaZrj5FLWVbndGAMGYg@mail.gmail.com>
Date: Wed, 20 Jul 2011 13:28:57 +0200
X-Google-Sender-Auth: bi0qXRgPBCpLFqGKWM2TfhD5UlY
Message-ID: <CADdV=MsXmL-nN4G+_ip2Y5wO8qQetOZcD-r+543vJNAHm2oLfg@mail.gmail.com>
Subject: Re: nits with new _git
From: Nikolai Weibull <now@bitwi.se>
To: Mikael Magnusson <mikachu@gmail.com>
Cc: zsh workers <zsh-workers@zsh.org>
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: quoted-printable

On Fri, Jun 3, 2011 at 23:39, Mikael Magnusson <mikachu@gmail.com> wrote:
> It looks like
> this is a systematic error in the whole _git file, but should be
> easily fixed. _git() itself defines local ret=3D1, then does
> _call_function ret _git-$words[1], then _git-log() in this case sets
> ret=3D0 on success. Sounds good? Not really, because it doesn't return
> ret, so then _call_function overwrites ret with something else. The
> solution would seem to simply not use _call_function since none of the
> helper functions return anything, they just set ret directly.

I=E2=80=99d rather remove all superfluous =E2=9F=A8&& ret=3D0=E2=9F=A9s and=
 add =E2=9F=A8integer ret=3D1=E2=9F=A9
where appropriate.

Opinions?