Re: [PATCH] config: test for --replace-all with one argument and fix documentation.
- From
Johannes Schindelin <johannes.schindelin@gmx.de>
- Date
- Mar 17, 2009, 10:24 UTC
- Message-ID
- <alpine.DEB.1.00.0903171123530.6393@intel-tinevez-2-302>
- In-Reply-To
- <1237283197.10001.9.camel@equipo-loli>
Hi,
On Tue, 17 Mar 2009, Carlos Rica wrote:
Show 5 quoted lines
> Option --replace-all only allows at least two arguments, so > documentation was needing to be updated accordingly. A test showing > that the command fails with only one parameter is also provided. > > Signed-off-by: Carlos Rica <jasampler@gmail.com>
Looks obviously correct to me. I am actually unsure if I can ACK this patch, as most of builtin-config.c does not look all that familiar to me anymore ;-)
Ciao, Dscho