From mboxrd@z Thu Jan  1 00:00:00 1970
Return-Path: <zsh-workers-return-30205-mason-zsh=primenet.com.au@zsh.org>
Received: (qmail 18146 invoked by alias); 11 Feb 2012 01:16:39 -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: 30205
Received: (qmail 2423 invoked from network); 11 Feb 2012 01:16:26 -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=-2.5 required=5.0 tests=BAYES_00,DKIM_SIGNED,
	RCVD_IN_DNSWL_LOW,T_DKIM_INVALID autolearn=ham version=3.3.2
Received-SPF: none (ns1.primenet.com.au: domain at daniel.shahaf.name does not designate permitted sender hosts)
DKIM-Signature: v=1; a=rsa-sha1; c=relaxed/relaxed; d=
	daniel.shahaf.name; h=date:from:to:cc:subject:message-id
	:references:mime-version:content-type:in-reply-to; s=mesmtp; bh=
	LYio+h3+pqdWtNPkVpvTw25A00I=; b=jcmcfXHdy3qBX8kZGKWjdJnoNx6Fqeg6
	5nmVuE2GcMrJGSOecUHVk2+dlx+uvYy6W5a7n0dnVOHl9KxkfCb2keVJTZVXXagp
	P3Ns2OOWqo9akZDsKFFmAwL45xmVKX6trgUkJBN4f+m1UoQ636Wjjx/u4EDPeDcJ
	iFmWGwAQ6Xk=
DKIM-Signature: v=1; a=rsa-sha1; c=relaxed/relaxed; d=
	messagingengine.com; h=date:from:to:cc:subject:message-id
	:references:mime-version:content-type:in-reply-to; s=smtpout;
	 bh=LYio+h3+pqdWtNPkVpvTw25A00I=; b=iA8icMZtGbKiJCLt20UXIa+hyA7j
	1DvL8GIPbbvklpsutdlgWyN103oSgtjhslJZAsGWE0/o3w9m3qD+KWz6d6K3L/v1
	SJ0sj3C1A2v5rm30+92aSEnB8TjjlfV7SPnQoQT39E6BP4WfBXoisoss8NDWjRwK
	lo1qEWA7IWzaYz0=
X-Sasl-enc: gZYLOGqt7w93cUV74ksxUU3O6SnsNGm6quBqNIH68bPQ/PYs8XnU6NVLffSeuA 1328922522
Date: Sat, 11 Feb 2012 03:08:31 +0200
From: Daniel Shahaf <d.s@daniel.shahaf.name>
To: Oliver Kiddle <okiddle@yahoo.co.uk>
Cc: Zsh workers <zsh-workers@zsh.org>
Subject: Re: PATCH: complete subversion change lists
Message-ID: <20120211010831.GA16088@daniel3.local>
References: <22654.1328888328@thecus.kiddle.eu>
MIME-Version: 1.0
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline
In-Reply-To: <22654.1328888328@thecus.kiddle.eu>
User-Agent: Mutt/1.5.20 (2009-06-14)

Oliver Kiddle wrote on Fri, Feb 10, 2012 at 16:38:48 +0100:
> +          while (( idx=$args[(I)*--c(l|hangelist):arg:] )); do
> +            args[(I)*--c(l|hangelist):arg:]=( \*{--cl,--changelist}':change list:_svn_changelists' )
> +          done

You could pass --quiet and/or --ignore-externals to save some disk
crawling.  (If --depth is already present on the command-line, you could
pass that too; but I don't know how easy it is to check that.)

In 1.7+, determining all valid changelist names can be done in a single
SQLite query.  Would it make sense to expose that in svn's API?

(Or you could run that query yourself --- but the SQLite schema is not
part of svn's public API and its backwards-compatibility guarantees.)