find . \( -name "*.cpp" -or -name "*.h" -or -name "*.c" -or -name "*.cc" \) -exec clang-format -i {} \;
If you reached this file doing a git blame, please see README.clang-format (added 2 commits in the future of this one)
Summary:
The "<video></video>" tag is a placeholder, and should only be processed
when it has been inserted for this toc item, i.e. when movieAnnots is
non-empty. Also place the search cursor at the last inserted block,
otherwise the expression will search from the beginning of the document.
Do likewise for audio/soundActions.
This makes loading large documents much fasters, and even fixes crashing
on huge documents, e.g. from project Gutenberg "The Complete Works of
William Shakespeare" (3200 pages) or the "King James Bible" (1400 pages).
Test Plan: open a large EPub file (see above)
Reviewers: #okular, aacid
Reviewed By: #okular, aacid
Subscribers: aacid, okular-devel
Tags: #okular
Differential Revision: https://phabricator.kde.org/D20948
Summary:
QTextDocument chokes badly when a some documents are wrapped inside
a table, returning e.g. a pagecount of -41292 afterwards.
On the downside, this removes any padding from the page. On the upside,
it removes any padding from the page.
BUG: 406738
BUG: 407140
CCBUG: 406116
Reviewers: #okular
Subscribers: okular-devel
Tags: #okular
Differential Revision: https://phabricator.kde.org/D20949
Summary:
There is no reason to fetch the palette once per loop,
just do it once
Reviewers: #okular, ngraham
Reviewed By: ngraham
Subscribers: okular-devel
Tags: #okular
Differential Revision: https://phabricator.kde.org/D20947
Summary:
The loop body is skipped when `epub_it_get_curr(it)` is invalid.
Make it more obvious there is no more code executed in this case.
Test Plan: no functional changes
Reviewers: #okular, ngraham
Reviewed By: ngraham
Subscribers: okular-devel
Tags: #okular
Differential Revision: https://phabricator.kde.org/D20946
Summary:
libepub can't handle whitespace url encodings. This is why epub documents containing whitespaces urls aren't
displayed correctly. Since libepub doesn't seem to be maintained anymore, it might be better to fix this issue here.
BUG: 334711
Test Plan: Example epubs are in the ticket.
Reviewers: #okular, aacid
Reviewed By: #okular, aacid
Subscribers: ngraham, aacid, okular-devel
Tags: #okular
Differential Revision: https://phabricator.kde.org/D14310
This patch changes multiline QStringLiterals to QString::fromLatin1
as QStringLiterals doesn't understand multiline strings on windows.
Also, strncasecmp is replaced by the portable qstrnicmp function provided
by QByteArray header.
REVIEW: 126379
Diggory i think there was a misunderstanding in our discussion
The patch is unfinished (not goot to commit with a TODO)
and needs to be improved (need to find a way to not break ABI, QSize param should be const &) please
open a reviewrequest and we can discuss/finalize it there.
CCMAIL: diggory.hardy@unibas.ch
this way we can load resources of 0 bytes (not much useful per-se, but at least we know they are actually existing in the epub archive)
svn path=/trunk/KDE/kdegraphics/okular/; revision=1182899
When iterating through the document HTML files, set in the epub document the URL of the current file being loaded,
so it is possible to resolve the locations of resources based on the base URL of the current file.
This makes it possible to load resources like "foo.css", "somedir/foo.css", "../somedir/foo.css" and so on.
BUG: 198764
FIXED-IN: 4.6
svn path=/trunk/KDE/kdegraphics/okular/; revision=1182629
It will now try to load toc elements which are not part of the spine and put them
in the end of the document.
Also toc element which points to nowhere, will now point to the first page
as okular mandate some block for addTitle
svn path=/trunk/KDE/kdegraphics/okular/; revision=795292