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

2016-02-26 Thread Aleix Pol Gonzalez
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/127042/ --- (Updated Feb. 26, 2016, 4:11 p.m.) Status -- This change has been

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

2016-02-26 Thread Albert Vaca Cintora
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/127042/#review92804 --- Ship it! Looks good to me. - Albert Vaca Cintora On

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

2016-02-24 Thread Aleix Pol Gonzalez
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/127042/ --- (Updated Feb. 24, 2016, 6:21 p.m.) Review request for Okular and Alex

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

2016-02-11 Thread Aleix Pol Gonzalez
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/127042/ --- Review request for Okular. Repository: okular Description ---

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

2016-02-11 Thread Alex Richardson
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/127042/#review92284 --- This will still fail when invoked via dbus right? maybe we

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

2016-02-11 Thread Aleix Pol Gonzalez
> On Feb. 12, 2016, 1:40 a.m., Alex Richardson wrote: > > This will still fail when invoked via dbus right? maybe we should just > > remove the strict mode, I have no idea why I added that. Well, it will work if the URL is strictly encoded. - Aleix