Don't return a reference to a local variable, and removed a redundant API.

llvm-svn: 180713

1 files changed, 3 insertions, 13 deletions

diff --git a/lldb/source/Host/common/FileSpec.cpp b/lldb/source/Host/common/FileSpec.cpp
index 4764169a618..8405c0a9f2b 100644
--- a/lldb/source/Host/common/FileSpec.cpp
+++ b/lldb/source/Host/common/FileSpec.cpp
@@ -702,12 +702,12 @@ FileSpec::GetPath(char *path, size_t path_max_len) const
     return 0;
 }
 
-void
-FileSpec::GetPath (std::string &path) const
+std::string
+FileSpec::GetPath (void) const
 {
+    std::string path;
     const char *dirname = m_directory.GetCString();
     const char *filename = m_filename.GetCString();
-    path.clear();
     if (dirname)
     {
         path.append (dirname);
@@ -715,17 +715,7 @@ FileSpec::GetPath (std::string &path) const
             path.append ("/");
     }
     if (filename)
-    {
         path.append (filename);
-    }
-}
-
-
-std::string&
-FileSpec::GetPath (void) const
-{
-    std::string path;
-    GetPath (path);
     return path;
 }