On Thu, Mar 11, 2021 at 12:04:40PM -0500, Taylor Blau wrote: > Now that there is a shared 'flags' member in the options structure, > there is no need to keep track of whether to force progress or not, > since ultimately the decision of whether or not to show a progress meter > is controlled by a bit in the flags member. Just going back to what I wrote for patch 1, I think this "now that there is a shared flags..." bit is what misled me. You can easily have done this patch by just manipulating the local "flags" variable. And the rationale for the patch is "we can get rid of opts.progress, because nobody ever reads it except to set a bit opts.flags". Definitely not worth re-rolling or anything; I'm just explaining my earlier comments. :) -Peff
