On Tue, Apr 2, 2019 at 1:45 AM Eric Sunshine <sunshine@xxxxxxxxxxxxxx> wrote: > On Mon, Apr 1, 2019 at 7:52 AM SZEDER Gábor <szeder.dev@xxxxxxxxx> wrote: > > + if (show_update) { > > [...] > > progress_update = 0; > > - return; > > } > > Removal of these two 'returns' is unrelated to the change made by this > patch and should have been done by 1/4. > > > return; > > Likewise, this final 'return' doesn't need to be here and should go away in 1/4. I forgot to mention that these are micro-nits, not necessarily worth a re-roll.