[Okular-devel] Review Request: make core/document.cpp compile without warning when QT_ASCII_CAST_WARNINGS is set
Friedrich W. H. Kossebau
kossebau at kde.org
Sat Aug 27 11:54:19 UTC 2011
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/102453/
-----------------------------------------------------------
Review request for Okular.
Summary
-------
Or: flood code with QLatin1String/QLatin1Char
Interested in such patches? Would do more to the other files if you welcome these changes.
Have not done any measurements if this brings real advantages, but some µ-improvements should be there by theory.
Style okay? Or should resulting very long lines be broken? And should there be spaces between '('/')' and '"' ?
Diffs
-----
CMakeLists.txt ac12273
core/document.cpp a417828
Diff: http://git.reviewboard.kde.org/r/102453/diff
Testing
-------
Could still load and navigate a PDF with a toc.
Thanks,
Friedrich W. H.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/okular-devel/attachments/20110827/4d94cf61/attachment.html>
More information about the Okular-devel
mailing list