diff options
| author |  Lioncash | 2021-04-20 12:57:45 -0400 |
|---|---|---|
| committer | Lioncash | 2021-04-20 12:57:49 -0400 |
| commit | 6125590a7b2bbad7d5efdfbab69ba86601e0769b (patch) | |
| tree | c420bcf8709cf5af735988fddf25bba63ddbd2eb /src/common/logging/backend.cpp | |
| parent | log/backend: Make use of erase_if (diff) | |
| download | yuzu-6125590a7b2bbad7d5efdfbab69ba86601e0769b.tar.gz yuzu-6125590a7b2bbad7d5efdfbab69ba86601e0769b.tar.xz yuzu-6125590a7b2bbad7d5efdfbab69ba86601e0769b.zip | |
log/backend: Use in-class initializer for FileBackend
We can also avoid redundant constructions of the same string repeatedly.
Diffstat (limited to 'src/common/logging/backend.cpp')
| -rw-r--r-- | src/common/logging/backend.cpp | 10 |
1 files changed, 6 insertions, 4 deletions
diff --git a/src/common/logging/backend.cpp b/src/common/logging/backend.cpp index f0bb392c6..96efa977d 100644 --- a/src/common/logging/backend.cpp +++ b/src/common/logging/backend.cpp | |||
| @@ -148,12 +148,14 @@ void ColorConsoleBackend::Write(const Entry& entry) { | |||
| 148 | PrintColoredMessage(entry); | 148 | PrintColoredMessage(entry); |
| 149 | } | 149 | } |
| 150 | 150 | ||
| 151 | FileBackend::FileBackend(const std::string& filename) : bytes_written(0) { | 151 | FileBackend::FileBackend(const std::string& filename) { |
| 152 | if (FS::Exists(filename + ".old.txt")) { | 152 | const auto old_filename = filename + ".old.txt"; |
| 153 | FS::Delete(filename + ".old.txt"); | 153 | |
| 154 | if (FS::Exists(old_filename)) { | ||
| 155 | FS::Delete(old_filename); | ||
| 154 | } | 156 | } |
| 155 | if (FS::Exists(filename)) { | 157 | if (FS::Exists(filename)) { |
| 156 | FS::Rename(filename, filename + ".old.txt"); | 158 | FS::Rename(filename, old_filename); |
| 157 | } | 159 | } |
| 158 | 160 | ||
| 159 | // _SH_DENYWR allows read only access to the file for other programs. | 161 | // _SH_DENYWR allows read only access to the file for other programs. |