Use std::filesystem to build task paths (#452)

aobolensk · web-flow · commit 58f945dde8eb · 2025-06-20T07:32:58.000Z
Use `std::filesystem::path` when constructing task file paths
diff --git a/modules/core/util/src/util.cpp b/modules/core/util/src/util.cpp
@@ -9,14 +9,15 @@
 namespace {
 
 std::string GetAbsolutePath(const std::string& relative_path) {
-  const std::filesystem::path path = std::string(PPC_PATH_TO_PROJECT) + "/tasks/" + relative_path;
+  std::filesystem::path path = std::filesystem::path(PPC_PATH_TO_PROJECT) / "tasks" / relative_path;
   return path.string();
 }
 
 }  // namespace
 
 std::string ppc::util::GetAbsoluteTaskPath(const std::string& id_path, const std::string& relative_path) {
-  return GetAbsolutePath(id_path + "/data/" + relative_path);
+  std::filesystem::path task_relative = std::filesystem::path(id_path) / "data" / relative_path;
+  return GetAbsolutePath(task_relative.string());
 }
 
 int ppc::util::GetNumThreads() {

Original file line number	Diff line number	Diff line change
`@@ -9,14 +9,15 @@`
`9`	`9`	`namespace {`
`10`	`10`
`11`	`11`	`std::string GetAbsolutePath(const std::string& relative_path) {`
`12`		`- const std::filesystem::path path = std::string(PPC_PATH_TO_PROJECT) + "/tasks/" + relative_path;`
	`12`	`+ std::filesystem::path path = std::filesystem::path(PPC_PATH_TO_PROJECT) / "tasks" / relative_path;`
`13`	`13`	`return path.string();`
`14`	`14`	`}`
`15`	`15`
`16`	`16`	`} // namespace`
`17`	`17`
`18`	`18`	`std::string ppc::util::GetAbsoluteTaskPath(const std::string& id_path, const std::string& relative_path) {`
`19`		`- return GetAbsolutePath(id_path + "/data/" + relative_path);`
	`19`	`+ std::filesystem::path task_relative = std::filesystem::path(id_path) / "data" / relative_path;`
	`20`	`+ return GetAbsolutePath(task_relative.string());`
`20`	`21`	`}`
`21`	`22`
`22`	`23`	`int ppc::util::GetNumThreads() {`