diff options
| author | Lioncash <mathew1800@gmail.com> | 2019-05-23 14:33:27 -0400 |
|---|---|---|
| committer | Lioncash <mathew1800@gmail.com> | 2019-05-23 14:33:29 -0400 |
| commit | e7ab0e91278c80eee110edda0c737ea05a4f0704 (patch) | |
| tree | 96714ad5c5c932b1ced407121ff2d47740a5ed78 /src/common/file_util.cpp | |
| parent | 11e9bee91d645cba69e936916394a0a03875c878 (diff) | |
common/file_util: Remove unnecessary return at end of void StripTailDirSlashes()
While we're at it, also invert the conditional into a guard clause.
Diffstat (limited to 'src/common/file_util.cpp')
| -rw-r--r-- | src/common/file_util.cpp | 14 |
1 files changed, 8 insertions, 6 deletions
diff --git a/src/common/file_util.cpp b/src/common/file_util.cpp index d8812837ea..2d9374783d 100644 --- a/src/common/file_util.cpp +++ b/src/common/file_util.cpp @@ -78,13 +78,15 @@ namespace FileUtil { // Remove any ending forward slashes from directory paths // Modifies argument. static void StripTailDirSlashes(std::string& fname) { - if (fname.length() > 1) { - std::size_t i = fname.length(); - while (i > 0 && fname[i - 1] == DIR_SEP_CHR) - --i; - fname.resize(i); + if (fname.length() <= 1) { + return; + } + + std::size_t i = fname.length(); + while (i > 0 && fname[i - 1] == DIR_SEP_CHR) { + --i; } - return; + fname.resize(i); } bool Exists(const std::string& filename) { |