Gentoo Logo
Gentoo Spaceship




Note: Due to technical difficulties, the Archives are currently not up to date. GMANE provides an alternative service for most mailing lists.
c.f. bug 424647
List Archive: gentoo-dev
Navigation:
Lists: gentoo-dev: < Prev By Thread Next > < Prev By Date Next >
Headers:
To: gentoo-dev@g.o
From: Ulrich Mueller <ulm@g.o>
Subject: Re: git-2: a bunch of patches to review
Date: Thu, 22 Sep 2011 09:01:17 +0200
> Attaching fixed version of the last two patches, and a complete
> eclass for convenience.

Just a general comment: Is it really necessary to change all
[[ -n ${foo} ]] and [[ -z ${foo} ]] conditionals to the more obscure
[[ ${foo} ]] and [[ ! ${foo} ]]?

The shortest possible form is not always the one that's best readable.

Ulrich


Replies:
Re: git-2: a bunch of patches to review
-- Michał Górny
Re: git-2: a bunch of patches to review
-- Fabian Groffen
References:
Re: git-2: a bunch of patches to review
-- Michał Górny
Navigation:
Lists: gentoo-dev: < Prev By Thread Next > < Prev By Date Next >
Previous by thread:
Re: git-2: a bunch of patches to review
Next by thread:
Re: git-2: a bunch of patches to review
Previous by date:
Re: finding reverse dependencies for arch testing (and other purposes)
Next by date:
Re: Please don't use IUSE=static-libs unless really necessary


Updated Jun 29, 2012

Summary: Archive of the gentoo-dev mailing list.

Donate to support our development efforts.

Copyright 2001-2013 Gentoo Foundation, Inc. Questions, Comments? Contact us.