[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: wget | Draft: Fix #60956 (!26)
From: |
@rockdaboot |
Subject: |
Re: wget | Draft: Fix #60956 (!26) |
Date: |
Sun, 08 Aug 2021 08:07:25 +0000 |
Tim Rühsen commented on a discussion:
https://gitlab.com/gnuwget/wget/-/merge_requests/26#note_645865137
Just one question, why do you think that "license" is relevant for displaying
the page ?
Thanks to your MR and the Savannah bug report, I could fix the code for Wget2
:smile:
```
while (len) {
const char *p;
for (p = val;len && !c_isspace(*val); val++, len--);
if (p == val) { val++; len--; continue; }
// Check for items that may be important to display the page.
if (!wget_strncasecmp_ascii(p, "icon", val - p)
|| !wget_strncasecmp_ascii(p, "manifest", val - p)
|| !wget_strncasecmp_ascii(p, "modulepreload", val - p)
|| !wget_strncasecmp_ascii(p, "stylesheet", val - p)
|| !wget_strncasecmp_ascii(p, "prefetch", val - p)
|| !wget_strncasecmp_ascii(p, "preload", val - p))
{
ctx->link_inline = 1;
break;
}
}
```
--
Reply to this email directly or view it on GitLab:
https://gitlab.com/gnuwget/wget/-/merge_requests/26#note_645865137
You're receiving this email because of your account on gitlab.com.
Re: wget | Draft: Fix #60956 (!26), @rockdaboot, 2021/08/21
Re: wget | Draft: Fix #60956 (!26), @rockdaboot, 2021/08/21
Re: wget | Draft: Fix #60956 (!26), Freddybobjo (@wadersgeek), 2021/08/26