Skip to content

Commit

Permalink
Merge pull request #45 from personalrobotics/bugfix/retrieverResolveURI
Browse files Browse the repository at this point in the history
Return nullptr instead of blank string
  • Loading branch information
mkoval committed Apr 27, 2016
2 parents 96e61c9 + 19238dd commit e0edcad
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions aikido/src/util/CatkinResourceRetriever.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -229,13 +229,13 @@ Uri CatkinResourceRetriever::resolvePackageUri(const Uri& _uri) const
using boost::filesystem::path;

if(!_uri.mScheme || *_uri.mScheme != "package")
return "";
return Uri();

if(!_uri.mAuthority)
{
dtwarn << "Failed extracting package name from URI '"
<< _uri.toString() << ".\n";
return "";
return Uri();
}

const std::string& packageName = *_uri.mAuthority;
Expand Down

0 comments on commit e0edcad

Please sign in to comment.