Re: [Wget-dev] wget2 | Handle wide characters in download progress bar. (!407)

[Darshit Shah]

Darshit Shah started a new discussion on libwget/bar.c:

>               // xxx.xxKB/s   _BAR_SPEED_SIZE         Download speed
>               // ===>         Remaining               Progress Meter
>  
> +             inspect_multibyte(slotp->filename, _BAR_FILENAME_SIZE, 
> &consumed, &pad);
>               wget_fprintf(stdout, "%-*.*s %*d%% [%s] %*s %*s%c/s",
> -                             _BAR_FILENAME_SIZE, _BAR_FILENAME_SIZE, 
> slotp->filename,
> +                             (int) (consumed+pad), (int) (consumed+pad), 
> slotp->filename,

Should these still not be `_BAR_FILENAME_SIZE`? The way I understand it, the 
sum should still always be a constant. Maybe, we can have an assert statement 
above to ensure this.

-- 
Reply to this email directly or view it on GitLab: 
https://gitlab.com/gnuwget/wget2/merge_requests/407#note_109178072
You're receiving this email because of your account on gitlab.com.