Fix bug returning nullptr in a string return type

j-rivero · scpeters · commit 5107000747d6 · 2025-06-06T20:59:18.000Z
Signed-off-by: Jose Luis Rivero <jrivero@honurobotics.com> (cherry picked from commit 53c9369)
diff --git a/src/gui/Gui.cc b/src/gui/Gui.cc
@@ -206,7 +206,7 @@ std::string defaultGuiConfigFile(bool _isPlayback,
       {
         gzerr << "Failed to create the default config folder ["
           << defaultConfigFolder << "]\n";
-        return nullptr;
+        return "";
       }
     }
 
@@ -217,7 +217,7 @@ std::string defaultGuiConfigFile(bool _isPlayback,
       gzerr << "Failed to copy installed config [" << installedConfig
              << "] to default config [" << defaultConfig << "]."
              << std::endl;
-      return nullptr;
+      return "";
     }
     else
     {

Original file line number	Diff line number	Diff line change
`@@ -206,7 +206,7 @@ std::string defaultGuiConfigFile(bool _isPlayback,`
`206`	`206`	`{`
`207`	`207`	`gzerr << "Failed to create the default config folder ["`
`208`	`208`	`<< defaultConfigFolder << "]\n";`
`209`		`- return nullptr;`
	`209`	`+ return "";`
`210`	`210`	`}`
`211`	`211`	`}`
`212`	`212`
`@@ -217,7 +217,7 @@ std::string defaultGuiConfigFile(bool _isPlayback,`
`217`	`217`	`gzerr << "Failed to copy installed config [" << installedConfig`
`218`	`218`	`<< "] to default config [" << defaultConfig << "]."`
`219`	`219`	`<< std::endl;`
`220`		`- return nullptr;`
	`220`	`+ return "";`
`221`	`221`	`}`
`222`	`222`	`else`
`223`	`223`	`{`