search

mame / wsl2-ssh-agent

A bridge from WSL2 ssh client to Windows ssh-agent.exe service
MIT License
102 stars 6 forks source link

Remove zero case for trial function #2

Closed jramosf closed 1 year ago

jramosf commented 1 year ago

The only two usages of the trial function come from range loop (starting at 0 on the waitTimes slice). Since we have (i+1) in the the Sprintf args, there is now no possibility to get into the i == 0 if branch.

This also leaves both usages with trial(i).

jramosf commented 1 year ago

@mame any chance this can get merged? Thanks!

mame commented 1 year ago

My intention was to not show (trial 1/3), and to show (trial 2/3) and (trial 3/3).Can you just change trial(i+1) to trial(i) and keep the definition of trial as is?

jramosf commented 1 year ago

Owner

Done, ready-to-review

mame commented 1 year ago

Thanks!