diff options
author | Christian Mollekopf <chrigi_1@fastmail.fm> | 2017-03-27 09:38:28 +0200 |
---|---|---|
committer | Christian Mollekopf <chrigi_1@fastmail.fm> | 2017-03-27 09:38:28 +0200 |
commit | f99a8084b875815dc18ae67eeaed3919e8249463 (patch) | |
tree | 5522f4353089e9a0775725129c8a152152ba14fe /examples/imapresource | |
parent | 8c27163cff4090abc9664eca1895931bc0bd14d1 (diff) | |
download | sink-f99a8084b875815dc18ae67eeaed3919e8249463.tar.gz sink-f99a8084b875815dc18ae67eeaed3919e8249463.zip |
Don't use 0 as error code and better error codes
Diffstat (limited to 'examples/imapresource')
-rw-r--r-- | examples/imapresource/imapresource.cpp | 2 | ||||
-rw-r--r-- | examples/imapresource/imapserverproxy.cpp | 4 | ||||
-rw-r--r-- | examples/imapresource/imapserverproxy.h | 1 |
3 files changed, 4 insertions, 3 deletions
diff --git a/examples/imapresource/imapresource.cpp b/examples/imapresource/imapresource.cpp index 5dce875..077795a 100644 --- a/examples/imapresource/imapresource.cpp +++ b/examples/imapresource/imapresource.cpp | |||
@@ -473,7 +473,7 @@ public: | |||
473 | { | 473 | { |
474 | if (error) { | 474 | if (error) { |
475 | if (error.errorCode == Imap::CouldNotConnectError) { | 475 | if (error.errorCode == Imap::CouldNotConnectError) { |
476 | return {ApplicationDomain::LoginError, error.errorMessage}; | 476 | return {ApplicationDomain::ConnectionError, error.errorMessage}; |
477 | } else if (error.errorCode == Imap::SslHandshakeError) { | 477 | } else if (error.errorCode == Imap::SslHandshakeError) { |
478 | return {ApplicationDomain::LoginError, error.errorMessage}; | 478 | return {ApplicationDomain::LoginError, error.errorMessage}; |
479 | } | 479 | } |
diff --git a/examples/imapresource/imapserverproxy.cpp b/examples/imapresource/imapserverproxy.cpp index a856576..0cc43b8 100644 --- a/examples/imapresource/imapserverproxy.cpp +++ b/examples/imapresource/imapserverproxy.cpp | |||
@@ -69,7 +69,7 @@ static KAsync::Job<T> runJob(KJob *job, const std::function<T(KJob*)> &f) | |||
69 | QObject::connect(job, &KJob::result, [&future, f](KJob *job) { | 69 | QObject::connect(job, &KJob::result, [&future, f](KJob *job) { |
70 | SinkTrace() << "Job done: " << job->metaObject()->className(); | 70 | SinkTrace() << "Job done: " << job->metaObject()->className(); |
71 | if (job->error()) { | 71 | if (job->error()) { |
72 | SinkWarning() << "Job failed: " << job->errorString(); | 72 | SinkWarning() << "Job failed: " << job->errorString() << job->metaObject()->className(); |
73 | future.setError(job->error(), job->errorString()); | 73 | future.setError(job->error(), job->errorString()); |
74 | } else { | 74 | } else { |
75 | future.setValue(f(job)); | 75 | future.setValue(f(job)); |
@@ -87,7 +87,7 @@ static KAsync::Job<void> runJob(KJob *job) | |||
87 | QObject::connect(job, &KJob::result, [&future](KJob *job) { | 87 | QObject::connect(job, &KJob::result, [&future](KJob *job) { |
88 | SinkTrace() << "Job done: " << job->metaObject()->className(); | 88 | SinkTrace() << "Job done: " << job->metaObject()->className(); |
89 | if (job->error()) { | 89 | if (job->error()) { |
90 | SinkWarning() << "Job failed: " << job->errorString(); | 90 | SinkWarning() << "Job failed: " << job->errorString() << job->metaObject()->className(); |
91 | future.setError(job->error(), job->errorString()); | 91 | future.setError(job->error(), job->errorString()); |
92 | } else { | 92 | } else { |
93 | future.setFinished(); | 93 | future.setFinished(); |
diff --git a/examples/imapresource/imapserverproxy.h b/examples/imapresource/imapserverproxy.h index cae3105..872f032 100644 --- a/examples/imapresource/imapserverproxy.h +++ b/examples/imapresource/imapserverproxy.h | |||
@@ -30,6 +30,7 @@ | |||
30 | namespace Imap { | 30 | namespace Imap { |
31 | 31 | ||
32 | enum ErrorCode { | 32 | enum ErrorCode { |
33 | NoError, | ||
33 | CouldNotConnectError, | 34 | CouldNotConnectError, |
34 | SslHandshakeError | 35 | SslHandshakeError |
35 | }; | 36 | }; |