Re: [PATCH] Add help.autocorrect to enable/disable autocorrecting

From

Johannes Schindelin <johannes.schindelin@gmx.de>

Date

Jul 23, 2008, 16:44 UTC

Message-ID

<alpine.DEB.1.00.0807231743030.8986@racer>

In-Reply-To

<20080722222510.GA4474@blimp.local>

Hi,

On Wed, 23 Jul 2008, Alex Riesen wrote:

> Johannes Schindelin, Tue, Jul 22, 2008 23:44:50 +0200:
> > However, I think that the intention of this patch is too much DWIMery, 
> > which might be good for me (just like my "git add remote" patch), but 
> > not for the general audience.
> 
> Mustn't be good for all

You meant "needn't"?  It is good for me ;-)

> And thanks for sharing.

You're welcome.

> +	n = 1;
> +	while (n < main_cmds.cnt &&
> +		best_similarity == similarity(main_cmds.names[n]->name))
> +		++n;

Mini-nit: you never ask for the value of n, only if it is 1 or larger.  So 
you do not need to count...