diff options
author | Simon Hausmann <simon.hausmann@theqtcompany.com> | 2014-12-15 23:58:36 +0100 |
---|---|---|
committer | Simon Hausmann <simon.hausmann@theqtcompany.com> | 2014-12-15 23:58:36 +0100 |
commit | f3ad790d987ed81c6c02ad41d77207b088d574c4 (patch) | |
tree | f264da659d244a93f7c8d615cad93e0df3c0039f /src/pdf/qpdfdocument.cpp | |
parent | b5e5c718fa1f7821d04ac684eb21003caa88b93d (diff) |
Clean up error handling
Diffstat (limited to 'src/pdf/qpdfdocument.cpp')
-rw-r--r-- | src/pdf/qpdfdocument.cpp | 24 |
1 files changed, 10 insertions, 14 deletions
diff --git a/src/pdf/qpdfdocument.cpp b/src/pdf/qpdfdocument.cpp index b6fd203..1a3a6f3 100644 --- a/src/pdf/qpdfdocument.cpp +++ b/src/pdf/qpdfdocument.cpp @@ -63,8 +63,12 @@ void QPdfDocumentPrivate::clear() asyncBuffer.open(QIODevice::ReadWrite); } -void QPdfDocumentPrivate::setErrorCode() +void QPdfDocumentPrivate::updateLastError() { + if (doc) { + lastError = QPdfDocument::NoError; + return; + } switch (FPDF_GetLastError()) { case FPDF_ERR_SUCCESS: lastError = QPdfDocument::NoError; break; case FPDF_ERR_UNKNOWN: lastError = QPdfDocument::UnknownError; break; @@ -96,10 +100,7 @@ QPdfDocument::Error QPdfDocumentPrivate::load(QIODevice *newDevice, bool transfe password = documentPassword.toUtf8(); doc = FPDF_LoadCustomDocument(this, password.constData()); - if (doc) - clearError(); - else - setErrorCode(); + updateLastError(); return lastError; } @@ -150,16 +151,11 @@ void QPdfDocumentPrivate::tryLoadDocument() Q_ASSERT(!doc); doc = FPDFAvail_GetDocument(avail, password); - if (doc) { - clearError(); - } else { - setErrorCode(); - if (lastError == QPdfDocument::IncorrectPasswordError) - emit q->passwordRequired(); - } - if (doc) { + updateLastError(); + if (lastError == QPdfDocument::IncorrectPasswordError) + emit q->passwordRequired(); + else if (doc) emit q->documentLoadStarted(); - } } void QPdfDocumentPrivate::checkComplete() |