On 12/10/2022 10:01, Ævar Arnfjörð Bjarmason wrote:
- const struct run_process_parallel_opts opts = { - .processes = processes, - - .get_next_task = get_next_task, - .start_failure = start_failure, - .task_finished = task_finished, - - .ungroup = run_processes_parallel_ungroup, - };
[...]
+ const struct run_process_parallel_opts opts = { + .processes = processes, + .ungroup = run_processes_parallel_ungroup, + + .get_next_task = get_next_task, + .start_failure = start_failure, + .task_finished = task_finished, + }; + + run_processes_parallel_1(&opts, pp_cb); }
I complained before that this is moving things around that you added in the last commit, but not only are you moving the designated initializer added in the last commit you are rearranging it.
Best Wishes Phillip