[PATCH v3 4/8] combine-diff: do not pass revs->dense_combined_merges redundantly

2014-09-06 Thread Thomas Rast
The existing code passed revs->dense_combined_merges along revs itself into the combine-diff functions, which is rather redundant. Remove the 'dense' argument until much further down the callchain to simplify callers. Note that while the caller in submodule.c needs to do extra work now, the next

Re: [PATCH v3 4/8] combine-diff: do not pass revs->dense_combined_merges redundantly

2014-09-08 Thread Junio C Hamano
Thomas Rast writes: > The existing code passed revs->dense_combined_merges along revs itself > into the combine-diff functions, which is rather redundant. Remove > the 'dense' argument until much further down the callchain to simplify > callers. It was not apparent that the changes to diff_tree

Re: [PATCH v3 4/8] combine-diff: do not pass revs->dense_combined_merges redundantly

2014-09-11 Thread Jens Lehmann
Am 08.09.2014 um 19:29 schrieb Junio C Hamano: Thomas Rast writes: The existing code passed revs->dense_combined_merges along revs itself into the combine-diff functions, which is rather redundant. Remove the 'dense' argument until much further down the callchain to simplify callers. It was