diff options
author | 2016-09-28 11:57:17 +1000 | |
---|---|---|
committer | 2016-09-28 11:57:17 +1000 | |
commit | e41e16cf5b2bc5c7255794ad92f73b162a2bfc29 (patch) | |
tree | 0d7206aa8d76d5411ab74ab87a52bd8af85e004a | |
parent | 6bc18458b8d9c7ee43d7cd42319d7c225c1e81a9 (diff) |
fix srcclean when doing multiple files in a single run (nw)
-rw-r--r-- | src/tools/srcclean.cpp | 18 |
1 files changed, 9 insertions, 9 deletions
diff --git a/src/tools/srcclean.cpp b/src/tools/srcclean.cpp index 9aac6ee845a..55ee72444fd 100644 --- a/src/tools/srcclean.cpp +++ b/src/tools/srcclean.cpp @@ -1874,7 +1874,7 @@ int main(int argc, char *argv[]) try { // instantiate appropriate cleaner implementation - char const *const ext(std::strrchr(argv[1], '.')); + char const *const ext(std::strrchr(argv[i], '.')); bool const is_c_file(ext && is_c_source_extension(ext)); bool const is_lua_file(ext && is_lua_source_extension(ext)); bool const is_xml_file(ext && is_xml_extension(ext)); @@ -1922,31 +1922,31 @@ int main(int argc, char *argv[]) if (!dry_run) { using namespace std::string_literals; - std::string const backup(argv[1] + ".orig"s); + std::string const backup(argv[i] + ".orig"s); std::remove(backup.c_str()); - if (std::rename(argv[1], backup.c_str())) + if (std::rename(argv[i], backup.c_str())) { - util::stream_format(std::cerr, "Error moving %1$s to backup location\n", argv[1]); + util::stream_format(std::cerr, "Error moving %1$s to backup location\n", argv[i]); ++failures; } else { - std::ofstream outfile(argv[1], std::ios_base::binary | std::ios_base::out | std::ios_base::trunc); + std::ofstream outfile(argv[i], std::ios_base::binary | std::ios_base::out | std::ios_base::trunc); outfile.write(&output[0], output.size()); outfile.flush(); if (!outfile) { - util::stream_format(std::cerr, "Error writing output to %1$s\n", argv[1]); + util::stream_format(std::cerr, "Error writing output to %1$s\n", argv[i]); ++failures; outfile.close(); - if (std::rename(backup.c_str(), argv[1])) - util::stream_format(std::cerr, "Error restoring backup of %1$s\n", argv[1]); + if (std::rename(backup.c_str(), argv[i])) + util::stream_format(std::cerr, "Error restoring backup of %1$s\n", argv[i]); } else if (!keep_backup) { if (std::remove(backup.c_str())) { - util::stream_format(std::cerr, "Error removing backup of %1$s\n", argv[1]); + util::stream_format(std::cerr, "Error removing backup of %1$s\n", argv[i]); ++failures; } } |