Don't clear whole line every time; instead, clear everything after end of line

[myndzi]

This makes the progress bar much less flickery on laggy or bursty connections over SSH, etc.

[jdan]

How does this work? Won't it clear whatever it just drew?

[myndzi]

Previously, when drawing an updated line, the code did this:

• clear the whole line
• go to the start of the line
• print the new line

Now, when drawing an updated line, the code does this:

• go to the start of the line
• print the new line
• clear the line from here to the end

(specifically, I'm making use of the "dir" argument in readline.clearLine: https://nodejs.org/api/readline.html#readline_readline_clearline_stream_dir)

The flicker resulted from pauses between steps 1 and 2 of the old method, and is eliminated by simply overwriting the previous characters so there is never a time when they aren't there.

[hallas]

From what I read it looks like it writes something, then clears the line from the point where the cursor is, clearing everything in between.

So your solution writes a new line on the old one and clears anything that might have been left over by the old write?

[myndzi]

Yes. Not sure what you mean by "in between" - note that the first step is "go to the beginning of the line"

[jdan]

Yep, way better. LGTM - thank you!

[hallas]

Nice change. Thanks.