[Okular-devel] Review Request: Replace deprecated QMatrix by QTransform
Albert Astals Cid
tsdgeos at terra.es
Tue Jul 31 21:47:37 UTC 2012
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/105737/#review16725
-----------------------------------------------------------
This changes public API (core/*.h) without any real reason so this is not going in, sorry. If you can you can provide a patch that marks the places of public api as // TODO change when doing an SIC change and that keeps the changes in non public code
- Albert Astals Cid
On July 26, 2012, 1:09 p.m., Jan Binder wrote:
>
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://git.reviewboard.kde.org/r/105737/
> -----------------------------------------------------------
>
> (Updated July 26, 2012, 1:09 p.m.)
>
>
> Review request for Okular.
>
>
> Description
> -------
>
> QMatrix is deprecated since Qt 4.6 according to Qt documentation.
> Luckily, QTransform acts as a more powerful drop-in replacement.
> This patch replaces all instances of QMatrix with QTransform and adds headers where necessary.
>
>
> Diffs
> -----
>
> core/annotations.h 23deaf3
> core/annotations.cpp bc55f4b
> core/annotations_p.h 36c3785
> core/area.h a11d4be
> core/area.cpp 94b4514
> core/page.cpp c778fe2
> core/page_p.h 02281f1
> core/rotationjob.cpp 625a5d2
> core/rotationjob_p.h 9790b64
> core/textpage.h a0b41c7
> core/textpage.cpp 57d0d0c
> core/textpage_p.h 7c2be22
> generators/spectre/rendererthread.cpp f185d22
> generators/xps/generator_xps.h b4fa594
> generators/xps/generator_xps.cpp 97f4450
>
> Diff: http://git.reviewboard.kde.org/r/105737/diff/
>
>
> Testing
> -------
>
> Okular still compiles without errors related to QMatrix or QTransform, document rotation still works, XPS documents still seem to work.
>
>
> Thanks,
>
> Jan Binder
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/okular-devel/attachments/20120731/8f8e4702/attachment.html>
More information about the Okular-devel
mailing list