Review Request: "data:" protocol: don't treat fragments as part of the data
David Faure
faure at kde.org
Tue Apr 17 20:47:08 BST 2012
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/104624/#review12612
-----------------------------------------------------------
kio/kio/dataprotocol.cpp
<http://git.reviewboard.kde.org/r/104624/#comment9863>
This looks wrong. If you call url.path(), you have a decoded string, so no need to call fromPercentEncoding on it.
kio/kio/dataprotocol.cpp
<http://git.reviewboard.kde.org/r/104624/#comment9864>
If data_offset is now always 0, this if can never pass, can it? Or maybe if raw_url_len is 0, but in any case this could be simplified to if (raw_url_len == 0)...
- David Faure
On April 16, 2012, 8:26 p.m., Rolf Eike Beer wrote:
>
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://git.reviewboard.kde.org/r/104624/
> -----------------------------------------------------------
>
> (Updated April 16, 2012, 8:26 p.m.)
>
>
> Review request for kdelibs.
>
>
> Description
> -------
>
> The testcases at http://greenbytes.de/tech/tc/datauri/ show that we incorrectly take the fragment (that's everything starting with the #) in a link containing a data URL as part of the URL. Fix it ;)
>
>
> Diffs
> -----
>
> kio/kio/dataprotocol.cpp ea95de4
> kio/tests/dataprotocoltest.cpp 6d7f1c7
>
> Diff: http://git.reviewboard.kde.org/r/104624/diff/
>
>
> Testing
> -------
>
> Old unittests still pass, new ones now also.
>
>
> Thanks,
>
> Rolf Eike Beer
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-core-devel/attachments/20120417/1ab0da6a/attachment.htm>
More information about the kde-core-devel
mailing list