On Tue, Apr 2, 2019 at 1:45 AM Eric Sunshine <sunsh...@sunshineco.com> wrote: > On Mon, Apr 1, 2019 at 7:52 AM SZEDER Gábor <szeder....@gmail.com> 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.