From 8e60f16821e57581172cf09c106dba6bbaf79384 Mon Sep 17 00:00:00 2001 From: "Uwe L. Korn" Date: Mon, 27 Oct 2014 20:12:56 +0100 Subject: [PATCH] Return type-safe nullptr --- src/libtomahawk/accounts/ResolverAccount.cpp | 14 +++++++------- 1 file changed, 7 insertions(+), 7 deletions(-) diff --git a/src/libtomahawk/accounts/ResolverAccount.cpp b/src/libtomahawk/accounts/ResolverAccount.cpp index 972df22ea..7711554a4 100644 --- a/src/libtomahawk/accounts/ResolverAccount.cpp +++ b/src/libtomahawk/accounts/ResolverAccount.cpp @@ -110,13 +110,13 @@ ResolverAccountFactory::createFromPath( const QString& path, const QString& fact if ( !( dir.exists() && dir.isReadable() ) ) //decompression fubar { displayError( tr( "Resolver installation error: cannot open bundle." ) ); - return 0; + return nullptr; } if ( !dir.cd( "content" ) ) //more fubar { displayError( tr( "Resolver installation error: incomplete bundle." ) ); - return 0; + return nullptr; } QString metadataFilePath = dir.absoluteFilePath( "metadata.json" ); @@ -128,7 +128,7 @@ ResolverAccountFactory::createFromPath( const QString& path, const QString& fact { dir.cdUp(); if ( !dir.cdUp() ) //we're in MANUALRESOLVERS_DIR - return 0; + return nullptr; QString name = configuration[ "pluginName" ].toString(); @@ -145,7 +145,7 @@ ResolverAccountFactory::createFromPath( const QString& path, const QString& fact configuration[ "bundleDir" ] = name; if ( !dir.cd( QString( "%1/content" ).arg( name ) ) ) //should work if it worked once - return 0; + return nullptr; } expandPaths( dir, configuration ); @@ -154,7 +154,7 @@ ResolverAccountFactory::createFromPath( const QString& path, const QString& fact if ( realPath.isEmpty() ) { displayError( tr( "Resolver installation error: bad metadata in bundle." ) ); - return 0; + return nullptr; } } else //either legacy resolver or uncompressed bundle, so we look for a metadata file @@ -190,7 +190,7 @@ ResolverAccountFactory::createFromPath( const QString& path, const QString& fact if ( !myPlatform.contains( platform ) ) { displayError( tr( "Resolver installation error: platform mismatch." ) ); - return 0; + return nullptr; } } @@ -203,7 +203,7 @@ ResolverAccountFactory::createFromPath( const QString& path, const QString& fact { displayError( tr( "Resolver installation error: Tomahawk %1 or newer is required." ) .arg( requiredVer ) ); - return 0; + return nullptr; } }