----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/130214/ -----------------------------------------------------------
(Updated Aug. 8, 2017, 12:56 p.m.) Status ------ This change has been marked as submitted. Review request for Okular. Changes ------- Submitted with commit 024d25cdeca11f9cd012b4e5099b1b4cb91826de by Oliver Sander to branch master. Repository: okular Description ------- Previously, they were drawn by modifying individual pixels of the QImage. With this patch we simply use a QPainter. That makes the code a lot shorter. This follows a suggestion from https://blog.hetzenecker.me/2017-06-30-gsoc-kdegraphics-okular/ The code that is removed by this patch is the sole consumer of the has_alpha variable. Consequently the compiler now warns about an unused variable. Should I send another patch removing that variable? Diffs ----- ui/pagepainter.cpp ecfafa502 Diff: https://git.reviewboard.kde.org/r/130214/diff/ Testing ------- Tested with pdf and odt documents, and different text colors, and I could not find any functional difference. In particular, testing with pdf and odt makes sure that both branches of the if(has_alpha) conditional are tested. Thanks, Oliver Sander