Re: [PATCH] Make XDF_NEED_MINIMAL default in blame.

2014-03-20 Thread Michael Andreen
On Thursday, March 20, 2014 01:45:21 PM Junio C Hamano wrote: > There hasn't been any argument for this patch, either. > > It is not like we are still in year 2007; timing result in a small > project like Git itself is not a good enough argument to change a > well established default at this late

[PATCH] Make XDF_NEED_MINIMAL default in blame.

2014-03-20 Thread Michael Andreen
er0m30.810s sys 0m0.684s (master)$ time PAGER=cat git blame --minimal -C -C -C -M cache.h > /dev/null real0m31.504s user 0m30.885s sys 0m0.598s Signed-off-by: Michael Andreen --- There hasn't been any arguments against this patch. Just updated the message with a note about --no

Re: [PATCH] Make XDF_NEED_MINIMAL default in blame.

2014-03-16 Thread Michael Andreen
On Sunday, March 16, 2014 01:12:01 PM Thomas Rast wrote: > Michael Andreen writes: > > > The --minimal flag is still there, but didn't want to break scripts > > depending on it. > > If I specify --no-minimal, does that turn it off again? > Yes, that works.

[PATCH] Make XDF_NEED_MINIMAL default in blame.

2014-03-16 Thread Michael Andreen
nimal -C -C -C -M cache.h > /dev/null real0m31.504s user 0m30.885s sys 0m0.598s Signed-off-by: Michael Andreen --- Additional measurements attached, the variation is fairly small. The --minimal flag is still there, but didn't want to break scripts depending on it. builtin/blam