diff options
author | Timothy Pearson <kb9vqf@pearsoncomputing.net> | 2012-01-21 13:17:48 -0600 |
---|---|---|
committer | Timothy Pearson <kb9vqf@pearsoncomputing.net> | 2012-01-21 13:17:48 -0600 |
commit | 292e3c830032a4634f04c3095d6bfa272b65d712 (patch) | |
tree | 37d103a34c4aebbbc76243461eb29b73152a916c /certmanager/lib/backends/qgpgme | |
parent | dc69027b0e7092ad69d8e58f1ee3aff22fd4d9c6 (diff) | |
download | tdepim-292e3c830032a4634f04c3095d6bfa272b65d712.tar.gz tdepim-292e3c830032a4634f04c3095d6bfa272b65d712.zip |
Fix a number of build warnings
Diffstat (limited to 'certmanager/lib/backends/qgpgme')
3 files changed, 16 insertions, 9 deletions
diff --git a/certmanager/lib/backends/qgpgme/qgpgmecryptoconfig.cpp b/certmanager/lib/backends/qgpgme/qgpgmecryptoconfig.cpp index d083947e9..053a8b924 100644 --- a/certmanager/lib/backends/qgpgme/qgpgmecryptoconfig.cpp +++ b/certmanager/lib/backends/qgpgme/qgpgmecryptoconfig.cpp @@ -661,13 +661,17 @@ void QGpgMECryptoConfigEntry::resetToDefault() { mSet = false; mDirty = true; - if ( mFlags & GPGCONF_FLAG_DEFAULT ) + if ( mFlags & GPGCONF_FLAG_DEFAULT ) { mValue = mDefaultValue; - else if ( mArgType == ArgType_None ) - if ( isList() ) + } + else if ( mArgType == ArgType_None ) { + if ( isList() ) { mValue = 0U; - else + } + else { mValue = false; + } + } } void QGpgMECryptoConfigEntry::setBoolValue( bool b ) diff --git a/certmanager/lib/backends/qgpgme/qgpgmerefreshkeysjob.cpp b/certmanager/lib/backends/qgpgme/qgpgmerefreshkeysjob.cpp index a17aca9e0..16846b59b 100644 --- a/certmanager/lib/backends/qgpgme/qgpgmerefreshkeysjob.cpp +++ b/certmanager/lib/backends/qgpgme/qgpgmerefreshkeysjob.cpp @@ -190,11 +190,14 @@ void Kleo::QGpgMERefreshKeysJob::slotProcessExited( KProcess * proc ) { if ( proc != mProcess ) return; - if ( !mError && !mPatternsToDo.empty() ) - if ( const GpgME::Error err = startAProcess() ) + if ( !mError && !mPatternsToDo.empty() ) { + if ( const GpgME::Error err = startAProcess() ) { mError = err; - else + } + else { return; + } + } emit done(); if ( !mError && diff --git a/certmanager/lib/backends/qgpgme/qgpgmesignencryptjob.cpp b/certmanager/lib/backends/qgpgme/qgpgmesignencryptjob.cpp index d5fa009db..1392afd5e 100644 --- a/certmanager/lib/backends/qgpgme/qgpgmesignencryptjob.cpp +++ b/certmanager/lib/backends/qgpgme/qgpgmesignencryptjob.cpp @@ -115,8 +115,8 @@ void Kleo::QGpgMESignEncryptJob::doOperationDoneEvent( const GpgME::Error & ) { } void Kleo::QGpgMESignEncryptJob::showErrorDialog( TQWidget * parent, const TQString & caption ) const { - if ( mResult.first.error() && !mResult.first.error().isCanceled() || - mResult.second.error() && !mResult.second.error().isCanceled() ) + if ( (mResult.first.error() && !mResult.first.error().isCanceled()) || + (mResult.second.error() && !mResult.second.error().isCanceled()) ) Kleo::MessageBox::error( parent, mResult.first, mResult.second, this, caption ); } |