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... Ciao, Dscho -- To unsubscribe from this list: send the line "unsubscribe git" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html