From: Junio C Hamano <gitster@pobox.com>
Date: Tue, 12 May 2015 22:14:59 GMT
Subject: Re: [PATCH 1/2] completion: add a helper function to get config variables
Message-ID: <xmqqa8x9e93w.fsf@gitster.dls.corp.google.com>
In-Reply-To: <1431262218-7304-1-git-send-email-szeder@ira.uka.de>

SZEDER Gábor <szeder@ira.uka.de> writes:

> +	for i in $(git --git-dir="$(__gitdir)" config --get-regexp "$section\..*" 2>/dev/null); do
>  		case "$i" in
> -		pretty.*)
> -			i="${i#pretty.}"
> +		$section.*)
> +			i="${i#$section.}"

The case arm treats $section as a shell glob, --get-regexp treats it
as a regex fragment and then the shell expansion uses it as a
literal.

The current set of callers give simple single-token like pretty,
remote, etc. to it so this is safe, but you may want to give a
comment to the function to help future generation, perhaps?

Other than that, looks quite straight-forward.

Thanks.