Web lists-archives.com

Re: [PATCH 1/2] Fix URL parsing




Hi Harald,

On 5/29/2016 11:30 AM, Harald Sitter wrote:
> You told it to construct a url from a local file path, so, that seems
> somewhat expected.
> 
> I'd presume what you should use is
> http://doc.qt.io/qt-5/qurl.html#fromUserInput which should usually
> make sense out of most random input.

Yes, I figured out that much. Thanks for the hint anyway.
I stared at the code for quite a while, but that never crossed my mind.
Missed the forest for the trees.

Anyway, discussion is happening on the review board:
~~
https://git.reviewboard.kde.org/r/128032/
~~

-- 
Regards,
Arno