Skip to content

Commit c99d457

Browse files
committed
Fix accidental use of function meant to remove() a file.
* `std::remove` is an `<algorithm>` * `remove()` is meant to remove a file (defined in `<stdio.h>`). Use `std::filsystem::remove()` instead. Signed-off-by: Henner Zeller <[email protected]>
1 parent 32ce8f2 commit c99d457

File tree

1 file changed

+4
-1
lines changed

1 file changed

+4
-1
lines changed

src/rmp/src/Restructure.cpp

Lines changed: 4 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -8,11 +8,13 @@
88
#include <unistd.h>
99

1010
#include <ctime>
11+
#include <filesystem>
1112
#include <fstream>
1213
#include <iostream>
1314
#include <limits>
1415
#include <mutex>
1516
#include <sstream>
17+
#include <system_error>
1618
#include <vector>
1719

1820
#include "base/abc/abc.h"
@@ -283,7 +285,8 @@ void Restructure::runABC()
283285

284286
for (const auto& file_to_remove : files_to_remove) {
285287
if (!logger_->debugCheck(RMP, "remap", 1)) {
286-
if (std::remove(file_to_remove.c_str()) != 0) {
288+
std::error_code err;
289+
if (std::filesystem::remove(file_to_remove, err); err) {
287290
logger_->error(RMP, 37, "Fail to remove file {}", file_to_remove);
288291
}
289292
}

0 commit comments

Comments
 (0)