From 7f8d21b4fc4484abf533b8da9fc9b99a6d0c73de Mon Sep 17 00:00:00 2001 From: "pkasting@chromium.org" Date: Mon, 18 Oct 2010 19:36:52 +0000 Subject: Convert LOG(INFO) to VLOG(1) - base/. Also removes an "else" after "return" and aligns "<<" per style guide. BUG=none TEST=none Review URL: http://codereview.chromium.org/3845002 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@62959 0039d316-1c4b-4281-b951-d872f2087c98 --- base/file_util_win.cc | 28 +++++++++++----------------- 1 file changed, 11 insertions(+), 17 deletions(-) (limited to 'base/file_util_win.cc') diff --git a/base/file_util_win.cc b/base/file_util_win.cc index d9f8404..30d314f 100644 --- a/base/file_util_win.cc +++ b/base/file_util_win.cc @@ -561,8 +561,7 @@ bool CreateTemporaryDirInDir(const FilePath& base_dir, FilePath path_to_create; srand(static_cast(time(NULL))); - int count = 0; - while (count < 50) { + for (int count = 0; count < 50; ++count) { // Try create a new temporary directory with random generated name. If // the one exists, keep trying another path name until we reach some limit. path_to_create = base_dir; @@ -572,17 +571,13 @@ bool CreateTemporaryDirInDir(const FilePath& base_dir, new_dir_name.append(base::IntToString16(rand() % kint16max)); path_to_create = path_to_create.Append(new_dir_name); - if (::CreateDirectory(path_to_create.value().c_str(), NULL)) - break; - count++; - } - - if (count == 50) { - return false; + if (::CreateDirectory(path_to_create.value().c_str(), NULL)) { + *new_dir = path_to_create; + return true; + } } - *new_dir = path_to_create; - return true; + return false; } bool CreateNewTempDirectory(const FilePath::StringType& prefix, @@ -600,14 +595,13 @@ bool CreateDirectory(const FilePath& full_path) { DWORD fileattr = ::GetFileAttributes(full_path_str); if (fileattr != INVALID_FILE_ATTRIBUTES) { if ((fileattr & FILE_ATTRIBUTE_DIRECTORY) != 0) { - DLOG(INFO) << "CreateDirectory(" << full_path_str << "), " - << "directory already exists."; + DVLOG(1) << "CreateDirectory(" << full_path_str << "), " + << "directory already exists."; return true; - } else { - LOG(WARNING) << "CreateDirectory(" << full_path_str << "), " - << "conflicts with existing file."; - return false; } + LOG(WARNING) << "CreateDirectory(" << full_path_str << "), " + << "conflicts with existing file."; + return false; } // Invariant: Path does not exist as file or directory. -- cgit v1.1