On Mon, Nov 15 2021, Junio C Hamano wrote: > * ab/only-single-progress-at-once (2021-11-03) 8 commits > - progress.c: add & assert a "global_progress" variable > - various *.c: use isatty(0|2), not isatty(STDIN_FILENO|STDERR_FILENO) > - pack-bitmap-write.c: don't return without stop_progress() > - progress.c: add temporary variable from progress struct > - progress.c tests: test some invalid usage > - progress.c tests: make start/stop commands on stdin > - progress.c test helper: add missing braces > - leak tests: fix a memory leaks in "test-progress" helper > > Further tweaks on progress API. It would be great to get this one marked for "next". I think all outstanding feedback was addressed ([1] for the last CL), and for a topic that's adding we haven't had any reports of the BUG() assertion it adds firing (and I've done some very thorough testing to make sure it won't, per the 8/8 commit message[2]). 1. https://lore.kernel.org/git/cover-v6-0.8-00000000000-20211102T122507Z-avarab@xxxxxxxxx/ 2. https://lore.kernel.org/git/patch-v6-8.8-bff919994b5-20211102T122507Z-avarab@xxxxxxxxx/
