[Okular-devel] Review Request 124026: Remove leftovers from KUrl -> QUrl port
Albert Astals Cid
aacid at kde.org
Thu Jul 9 21:20:47 UTC 2015
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/124026/#review82296
-----------------------------------------------------------
Why are you removing the test? Seems like a value test to me to make sure we're differentiating when # is a char of the file and when # refers to a page number.
- Albert Astals Cid
On jun. 8, 2015, 8:59 a.m., Jan Kundrát wrote:
>
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/124026/
> -----------------------------------------------------------
>
> (Updated jun. 8, 2015, 8:59 a.m.)
>
>
> Review request for Okular.
>
>
> Repository: okular
>
>
> Description
> -------
>
> When this test was written, converting from user-supplied strings to
> URLs with support for guessing whether the URL represents a local file
> (but only if it exists) or a remote URL was something not provided by
> Qt. That has changed in Qt 5.4, and what we have now is a unit test
> which is too stupid because it blindly concatenates cwd with a
> user-provided input.
>
> My understanding is that pre-5.4 Qt is supported on a best-effort basis,
> and that there's little point it carrying unit tests which only
> duplicate Qt's native features. Especially when these unit tests fail.
>
>
> Diffs
> -----
>
> autotests/CMakeLists.txt PRE-CREATION
> autotests/shelltest.cpp 0f750ae1f816e523f1bfaa8c19b9c947f0d0f8f3
> shell/okular_main.cpp b66b6a5f7cd30e6d123ba0b0a2d871e11807ea15
> shell/shellutils.h d5518c344c47dd81033a883cc43cca387764d362
> shell/shellutils.cpp 0dd737cf70ac888225a919799bd71905cfb89580
>
> Diff: https://git.reviewboard.kde.org/r/124026/diff/
>
>
> Testing
> -------
>
> I have no idea whether this introduces any regressions. There's plenty of test failures, both on the official KDE CI platform *and* on my local dev box. How do I, for example, persuade the KService/whatever to ignore the systemwide installation of Okular and to use, e.g., poppler generator from the build directory?
>
>
> Thanks,
>
> Jan Kundrát
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/okular-devel/attachments/20150709/2d5fb9c2/attachment-0001.html>
More information about the Okular-devel
mailing list