diff options
| author | Stefan Beller <sbeller@google.com> | 2016-03-01 05:07:16 +0300 |
|---|---|---|
| committer | Junio C Hamano <gitster@pobox.com> | 2016-03-01 22:57:19 +0300 |
| commit | aa71049485c7b789fb9121c7fc2bece431bc589b (patch) | |
| tree | f4239d8d4837cb6ae33083921709129693840340 /run-command.c | |
| parent | 2dac9b5637abe580e8b8ea36e00dbbf28ec99af6 (diff) | |
run_processes_parallel: rename parameters for the callbacks
The refs code has a similar pattern of passing around 'struct strbuf *err',
which is strictly used for error reporting. This is not the case here,
as the strbuf is used to accumulate all the output (whether it is error
or not) for the user. Rename it to 'out'.
Suggested-by: Jonathan Nieder <jrnieder@gmail.com>
Reviewed-by: Jonathan Nieder <jrnieder@gmail.com>
Signed-off-by: Stefan Beller <sbeller@google.com>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
Diffstat (limited to 'run-command.c')
| -rw-r--r-- | run-command.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/run-command.c b/run-command.c index bdda94006e..62c6721bd7 100644 --- a/run-command.c +++ b/run-command.c @@ -902,7 +902,7 @@ struct parallel_processes { struct strbuf buffered_output; /* of finished children */ }; -static int default_start_failure(struct strbuf *err, +static int default_start_failure(struct strbuf *out, void *pp_cb, void *pp_task_cb) { @@ -910,7 +910,7 @@ static int default_start_failure(struct strbuf *err, } static int default_task_finished(int result, - struct strbuf *err, + struct strbuf *out, void *pp_cb, void *pp_task_cb) { |