From 85938804a4a043c75cc4ca51a64ee789e55ba7cb Mon Sep 17 00:00:00 2001 From: Paul Cristian Sarbu Date: Thu, 6 Jul 2023 14:29:27 +0200 Subject: filesystem: Avoid unwanted indirections... ...that std::filesystem::* calls produce. This is because existence and type checks use almost exclusively std::filesystem::status, which follows symbolic links, when being called with path arguments. Instead, one should instead use these methods with the value returned by a call of std::filesystem::symlink_status. This commit also streamlines the FileSystemManager tests, as well as replace bare calls to std::filesystem with their FileSystemManager counterparts (where suitable). --- src/buildtool/main/main.cpp | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) (limited to 'src/buildtool/main/main.cpp') diff --git a/src/buildtool/main/main.cpp b/src/buildtool/main/main.cpp index 846fe257..8d2d4d2e 100644 --- a/src/buildtool/main/main.cpp +++ b/src/buildtool/main/main.cpp @@ -479,7 +479,7 @@ void SetupHashFunction() { -> Configuration { Configuration config{}; if (not clargs.config_file.empty()) { - if (not std::filesystem::exists(clargs.config_file)) { + if (not FileSystemManager::Exists(clargs.config_file)) { Logger::Log(LogLevel::Error, "Config file {} does not exist.", clargs.config_file.string()); -- cgit v1.2.3