Junio C Hamano <gitster@xxxxxxxxx> writes: > This step is meant to be just preparing by extending the interface > and passing the new argument throughout the callchain, I believe, > and it looks correctly done, assuming that we are OK to take this > "add a separate 'progress' parameter, when we need more parameters > later, the interface will gain more and more parameters" approach. After re-reading the remainder of the series, I think the structure this patch series takes (i.e. adding a new parameter to these callees) is better than the alternative (i.e. making sure everybody takes the pointer to the opts structure). Thanks. I couldn't review the proposed log messages of these commits, which were unreadable, at all, though.
