From: Nguyen Thai Ngoc Duy Date: Sat, 19 May 2012 04:56:40 GMT Subject: Re: [PATCH v2 1/5] Add convenient function to do automatic garbage collection Message-ID: In-Reply-To: On Sat, May 19, 2012 at 5:37 AM, Junio C Hamano wrote: > Nguyễn Thái Ngọc Duy   writes: > >> This function also avoids forking most of the time by performing some >> check in process. >> >> Signed-off-by: Nguyễn Thái Ngọc Duy >> ... >> @@ -64,6 +66,10 @@ static int gc_config(const char *var, const char *value, void *cb) >>               } >>               return git_config_string(&prune_expire, var, value); >>       } >> +     if (cb && !strcmp(var, cb)) { >> +             auto_gc = git_config_bool(var, value); >> +             return 0; >> +     } > > This does not look like "add convenient function for auto-gc" nor "avoid > forking"; it is something else that is not explained. It takes care of "foo.autogc" config key in commands that allows to suppress auto gc. But yes, commit message can use some improvement. -- Duy