Re: [PATCH v6 3/8] convert: Split start_multi_file_filter into two separate functions

2017-04-24 Thread Ben Peart
On 4/24/2017 12:31 AM, Junio C Hamano wrote: Ben Peart <peart...@gmail.com> writes: Subject: [PATCH v6 3/8] convert: Split start_multi_file_filter into two separate functions Two minor nits, because the capital after ":" looks ugly in shortlog output, and having () there

Re: [PATCH v6 3/8] convert: Split start_multi_file_filter into two separate functions

2017-04-23 Thread Junio C Hamano
Ben Peart <peart...@gmail.com> writes: > Subject: [PATCH v6 3/8] convert: Split start_multi_file_filter into two > separate functions Two minor nits, because the capital after ":" looks ugly in shortlog output, and having () there makes it easier to notice that a

[PATCH v6 3/8] convert: Split start_multi_file_filter into two separate functions

2017-04-21 Thread Ben Peart
To enable future reuse of the filter..process infrastructure, split start_multi_file_filter into two separate parts. start_multi_file_filter will now only contain the generic logic to manage the creation and tracking of the child process in a hashmap. start_multi_file_filter_fn is a protocol