[Webkit-unassigned] [Bug 221206] [SOUP] Stop using SoupBuffer in preparation for libsoup3

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Mon Feb 1 08:19:41 PST 2021


https://bugs.webkit.org/show_bug.cgi?id=221206

Adrian Perez <aperez at igalia.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |aperez at igalia.com

--- Comment #4 from Adrian Perez <aperez at igalia.com> ---
Comment on attachment 418874
  --> https://bugs.webkit.org/attachment.cgi?id=418874
Patch

LGTM as well

View in context: https://bugs.webkit.org/attachment.cgi?id=418874&action=review

>> Source/WebCore/platform/network/soup/ResourceRequestSoup.cpp:69
>> +        SoupBuffer* soupBuffer = soup_buffer_new_with_owner(g_bytes_get_data(data, nullptr),
> 
> auto* soupBuffer ?
> 
> Also I guess this would be migrated later right? Consider adding a comment.

I assume that there will be later a patch adding the support to actually
use libsoup3 inside “#if SOUP_CHECK_VERSION(2, 91, 0)” guards (or similar) and
this version of the code will be in the “#else” arm of the conditional =]

-- 
You are receiving this mail because:
You are the assignee for the bug.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.webkit.org/pipermail/webkit-unassigned/attachments/20210201/225e420d/attachment.htm>


More information about the webkit-unassigned mailing list