[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: wget2 | html_url.c: fixed srcset data urls in html_get_url() (!483)
From: |
Michael Roosz |
Subject: |
Re: wget2 | html_url.c: fixed srcset data urls in html_get_url() (!483) |
Date: |
Tue, 29 Dec 2020 22:13:38 +0000 |
Michael Roosz commented on a discussion on libwget/html_url.c:
https://gitlab.com/gnuwget/wget2/-/merge_requests/483#note_475515264
>
> for (;len && c_isspace(*val); val++,
> len--); // skip leading spaces
> for (p = val;len && !c_isspace(*val) &&
> *val != ','; val++, len--); // find end of URL
> +
> if (p != val) {
> +
> + if (len &&
> !wget_strncasecmp_ascii(p, "data:", 5)) { // ignore comma in Data URLs (see
> https://en.wikipedia.org/wiki/Data_URI_scheme)
> + for (val++, len--;len
> && !c_isspace(*val) && *val != ','; val++, len--); // find end of URL
Good point. Makes sense. I will create a unit-test, but might take a few days.
--
Reply to this email directly or view it on GitLab:
https://gitlab.com/gnuwget/wget2/-/merge_requests/483#note_475515264
You're receiving this email because of your account on gitlab.com.