diff options
| -rw-r--r-- | base/file_util_posix.cc | 9 | ||||
| -rw-r--r-- | base/test_file_util_posix.cc | 7 |
2 files changed, 6 insertions, 10 deletions
diff --git a/base/file_util_posix.cc b/base/file_util_posix.cc index 85cdc10..6d3984e 100644 --- a/base/file_util_posix.cc +++ b/base/file_util_posix.cc @@ -142,7 +142,7 @@ bool CopyDirectory(const FilePath& from_path, std::string suffix(&ent->fts_path[from_path.value().size()]); // Strip the leading '/' (if any). if (!suffix.empty()) { - DCHECK_EQ('/', suffix[0]); + DCHECK(suffix[0] == '/'); suffix.erase(0, 1); } const FilePath target_path = to_path.Append(suffix); @@ -157,7 +157,6 @@ bool CopyDirectory(const FilePath& from_path, } // Try creating the target dir, continuing on it if it exists already. - // Rely on the user's umask to produce correct permissions. if (mkdir(target_path.value().c_str(), 0777) != 0) { if (errno != EEXIST) error = errno; @@ -186,12 +185,10 @@ bool CopyDirectory(const FilePath& from_path, break; case FTS_SL: // Symlink. case FTS_SLNONE: // Symlink with broken target. - LOG(WARNING) << "CopyDirectory() skipping symbolic link: " << - ent->fts_path; + LOG(WARNING) << "CopyDirectory() skipping symbolic link."; continue; case FTS_DEFAULT: // Some other sort of file. - LOG(WARNING) << "CopyDirectory() skipping file of unknown type: " << - ent->fts_path; + LOG(WARNING) << "CopyDirectory() skipping weird file."; continue; default: NOTREACHED(); diff --git a/base/test_file_util_posix.cc b/base/test_file_util_posix.cc index 38aaae2..f55421c 100644 --- a/base/test_file_util_posix.cc +++ b/base/test_file_util_posix.cc @@ -44,14 +44,13 @@ bool CopyRecursiveDirNoCache(const std::wstring& source_dir, std::string suffix(&ent->fts_path[from_path.value().size()]); // Strip the leading '/' (if any). if (!suffix.empty()) { - DCHECK_EQ('/', suffix[0]); + DCHECK(suffix[0] == '/'); suffix.erase(0, 1); } const FilePath target_path = to_path.Append(suffix); switch (ent->fts_info) { case FTS_D: // Preorder directory. // Try creating the target dir, continuing on it if it exists already. - // Rely on the user's umask to produce correct permissions. if (mkdir(target_path.value().c_str(), 0777) != 0) { if (errno != EEXIST) error = errno; @@ -88,10 +87,10 @@ bool CopyRecursiveDirNoCache(const std::wstring& source_dir, break; case FTS_SL: // Symlink. case FTS_SLNONE: // Symlink with broken target. - LOG(WARNING) << "skipping symbolic link: " << ent->fts_path; + LOG(WARNING) << "skipping symbolic link."; continue; case FTS_DEFAULT: // Some other sort of file. - LOG(WARNING) << "skipping file of unknown type: " << ent->fts_path; + LOG(WARNING) << "skipping weird file."; continue; default: NOTREACHED(); |