From ca73f033344a67a915985b5d232d3abdcdbdc99e Mon Sep 17 00:00:00 2001 From: Peter Nelson Date: Fri, 12 Apr 2024 14:32:35 +0100 Subject: [PATCH] Codechange: Use std::filesystem::remove/rename in settingsgen. (#12483) --- src/settingsgen/settingsgen.cpp | 14 +++++--------- 1 file changed, 5 insertions(+), 9 deletions(-) diff --git a/src/settingsgen/settingsgen.cpp b/src/settingsgen/settingsgen.cpp index 4b265276bb..e2b0362b90 100644 --- a/src/settingsgen/settingsgen.cpp +++ b/src/settingsgen/settingsgen.cpp @@ -15,10 +15,7 @@ #include "../core/mem_func.hpp" #include "../error_func.h" -#if !defined(_WIN32) || defined(__CYGWIN__) -#include -#include -#endif +#include #include "../safeguards.h" @@ -493,15 +490,14 @@ int CDECL main(int argc, char *argv[]) AppendFile(after_file, fp); fclose(fp); + std::error_code error_code; if (CompareFiles(tmp_output, output_file)) { /* Files are equal. tmp2.xxx is not needed. */ - unlink(tmp_output); + std::filesystem::remove(tmp_output, error_code); } else { /* Rename tmp2.xxx to output file. */ -#if defined(_WIN32) - unlink(output_file); -#endif - if (rename(tmp_output, output_file) == -1) FatalError("rename() failed"); + std::filesystem::rename(tmp_output, output_file, error_code); + if (error_code) FatalError("rename({}, {}) failed: {}", tmp_output, output_file, error_code.message()); } } return 0;