refactor: use fs::weakly_canonical instead of current func

Author: Samat Gaynutdinov

server/src/Paths.cpp

@@ -108,11 +108,6 @@ namespace Paths {
         return fs::exists(path2) ? path2 : path1;
     }
 
-    fs::path getCanonicalPathIfExists(const std::string & filename) {
-        fs::path p = fs::weakly_canonical(filename);
-        return fs::exists(p) ? p : fs::path(filename);
-    }
-
     bool isPath(const std::string &possibleFilePath) noexcept {
         try {
             return fs::exists(possibleFilePath);

server/src/Paths.h

@@ -181,7 +181,6 @@ namespace Paths {
     }
 
     fs::path getCCJsonFileFullPath(const std::string &filename, const fs::path &directory);
-    fs::path getCanonicalPathIfExists(const std::string &filename);
 
     bool isPath(const std::string &possibleFilePath) noexcept;
     //endregion

server/src/printers/NativeMakefilePrinter.cpp

@@ -681,7 +681,7 @@ namespace printer {
 
     fs::path NativeMakefilePrinter::getRelativePath(fs::path source, bool isCanonical) const {
         if (isCanonical) {
-            source = Paths::getCanonicalPathIfExists(source);
+            source = fs::weakly_canonical(source);
         }
 
         for (const auto &[path, shellVariable] : pathToShellVariable) {
@@ -714,9 +714,8 @@ namespace printer {
                               std::function<void(const std::string&, const std::string&)> declareFunc,
                               bool shouldWriteToMap, bool isCanonical) {
         const fs::path relativePath = getRelativePath(path, isCanonical);
-        fs::path canonical = Paths::getCanonicalPathIfExists(path);
         if (isCanonical) {
-            path = canonical;
+            path = fs::weakly_canonical(path);
         }
         declareFunc(variableName, relativePath.string());
         if (shouldWriteToMap) {