wget-dev
[Top][All Lists]
Advanced

[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.




reply via email to

[Prev in Thread] Current Thread [Next in Thread]