[Okular-devel] Review Request 127042: Fix QUrl string encoding

Aleix Pol Gonzalez aleixpol at kde.org
Thu Feb 11 16:19:56 UTC 2016


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/127042/
-----------------------------------------------------------

Review request for Okular.


Repository: okular


Description
-------

This Url is converted in to a string to be parsed back using `QUrl( urlString, QUrl::StrictMode )`.
By default, it uses `PrettyDecoded`, which makes `QUrl::StrictMode` choke.

An alternative patch would be to overload openDocument to also take QUrl and skip the back-and-forth conversion.


Diffs
-----

  shell/okular_main.cpp 9bdc1fe 

Diff: https://git.reviewboard.kde.org/r/127042/diff/


Testing
-------

Now I can open files with a space in it.


Thanks,

Aleix Pol Gonzalez

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/okular-devel/attachments/20160211/e5e1aa5c/attachment.html>


More information about the Okular-devel mailing list