Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(2515)

Unified Diff: base/file_util_posix.cc

Issue 495002: Changes to base/ from a combination of FreeBSD and OpenBSD patches. (Closed)
Patch Set: minor tweaks Created 11 years ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: base/file_util_posix.cc
diff --git a/base/file_util_posix.cc b/base/file_util_posix.cc
index ed5abcca41e2f10df273cc0401dc8b4c9698e450..966c2b63e5117d376fb345b3f49731ad51c38107 100644
--- a/base/file_util_posix.cc
+++ b/base/file_util_posix.cc
@@ -85,7 +85,7 @@ int CountFilesCreatedAfter(const FilePath& path,
DIR* dir = opendir(path.value().c_str());
if (dir) {
-#if !defined(OS_LINUX) && !defined(OS_MACOSX) && !defined(OS_FREEBSD)
+#if !defined(OS_POSIX)
agl 2009/12/14 19:31:26 This is defeating the intent of the #ifdef which i
#error Depending on the definition of struct dirent, additional space for \
pathname may be needed
#endif
@@ -630,7 +630,7 @@ bool FileEnumerator::ReadDirectory(std::vector<DirectoryEntryInfo>* entries,
if (!dir)
return false;
-#if !defined(OS_LINUX) && !defined(OS_MACOSX) && !defined(OS_FREEBSD)
+#if !defined(OS_POSIX)
agl 2009/12/14 19:31:26 Likewise.
#error Depending on the definition of struct dirent, additional space for \
pathname may be needed
#endif

Powered by Google App Engine
This is Rietveld 408576698