aboutsummaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorMichael J. Spencer <bigcheesegs@gmail.com>2011-12-12 06:04:01 +0000
committerMichael J. Spencer <bigcheesegs@gmail.com>2011-12-12 06:04:01 +0000
commitc3b00e80400d27d5d6152374d87c0ad5866c780c (patch)
tree3adf92cdfd9478aae132d0d0d2ab5f25b57a79ed
parent1dd2ee7bf4d848e368829cb01033f297afb674ab (diff)
Support/FileSystem: Implement canonicalize.
git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@146363 91177308-0d34-0410-b5e6-96231b3b80d8
-rw-r--r--include/llvm/Support/FileSystem.h8
-rw-r--r--lib/Support/Unix/PathV2.inc7
-rw-r--r--lib/Support/Windows/PathV2.inc38
-rw-r--r--lib/Support/Windows/Windows.h20
-rw-r--r--unittests/Support/Path.cpp28
5 files changed, 99 insertions, 2 deletions
diff --git a/include/llvm/Support/FileSystem.h b/include/llvm/Support/FileSystem.h
index 42e61dcba7..6db4554aea 100644
--- a/include/llvm/Support/FileSystem.h
+++ b/include/llvm/Support/FileSystem.h
@@ -370,8 +370,12 @@ error_code unique_file(const Twine &model, int &result_fd,
/// @brief Canonicalize path.
///
-/// Sets result to the file system's idea of what path is. The result is always
-/// absolute and has the same capitalization as the file system.
+/// Sets result to the file system's idea of what path is. Path must be
+/// absolute. The result has the same case as the file system.
+///
+/// Example: Give a file system with "C:\a\b\c\file.txt".
+///
+/// C:\A\b\C\fIlE.TxT => C:\a\b\c\file.txt
///
/// @param path Input path.
/// @param result Set to the canonicalized version of \a path.
diff --git a/lib/Support/Unix/PathV2.inc b/lib/Support/Unix/PathV2.inc
index 7477b4f2a0..41ed49fec7 100644
--- a/lib/Support/Unix/PathV2.inc
+++ b/lib/Support/Unix/PathV2.inc
@@ -439,6 +439,13 @@ rety_open_create:
return success;
}
+error_code canonicalize(const Twine &path, SmallVectorImpl<char> &result) {
+ // Paths are already canonicalized on posix systems.
+ assert(path::is_absolute(path) && "path must be absolute!");
+ path.toVector(result);
+ return success;
+}
+
error_code detail::directory_iterator_construct(detail::DirIterState &it,
StringRef path){
SmallString<128> path_null(path);
diff --git a/lib/Support/Windows/PathV2.inc b/lib/Support/Windows/PathV2.inc
index afb5533dc8..abb53c2ce1 100644
--- a/lib/Support/Windows/PathV2.inc
+++ b/lib/Support/Windows/PathV2.inc
@@ -638,6 +638,44 @@ retry_create_file:
return success;
}
+error_code canonicalize(const Twine &path, SmallVectorImpl<char> &result) {
+ assert(path::is_absolute(path) && "path must be absolute!");
+ SmallString<128> path_storage;
+ StringRef p = path.toStringRef(path_storage);
+ SmallVector<wchar_t, 128> path_utf16;
+ result.set_size(0);
+
+ // Convert path to UTF-16.
+ if (error_code ec = UTF8ToUTF16(p, path_utf16))
+ return ec;
+
+ DWORD size = ::GetShortPathNameW(c_str(path_utf16), NULL, 0);
+ SmallVector<wchar_t, 128> short_path;
+ short_path.reserve(size + 1);
+ size = ::GetShortPathNameW( c_str(path_utf16)
+ , short_path.data()
+ , short_path.capacity());
+ if (!size)
+ return windows_error(::GetLastError());
+
+ short_path.set_size(size);
+
+ size = ::GetLongPathNameW(c_str(short_path), NULL, 0);
+ path_utf16.reserve(size + 1);
+ size = ::GetLongPathNameW( c_str(short_path)
+ , path_utf16.data()
+ , path_utf16.capacity());
+ if (!size)
+ return windows_error(::GetLastError());
+
+ path_utf16.set_size(size);
+
+ if (error_code ec = UTF16ToUTF8(path_utf16.data(), path_utf16.size(), result))
+ return ec;
+
+ return success;
+}
+
error_code get_magic(const Twine &path, uint32_t len,
SmallVectorImpl<char> &result) {
SmallString<128> path_storage;
diff --git a/lib/Support/Windows/Windows.h b/lib/Support/Windows/Windows.h
index 5c1da0d617..2754075035 100644
--- a/lib/Support/Windows/Windows.h
+++ b/lib/Support/Windows/Windows.h
@@ -128,6 +128,24 @@ struct FindHandleTraits : CommonHandleTraits {
}
};
+struct FileMappingHandleTraits : CommonHandleTraits {
+ static handle_type GetInvalid() {
+ return 0;
+ }
+};
+
+struct MappedViewOfFileHandleTraits : CommonHandleTraits {
+ typedef LPVOID handle_type;
+
+ static handle_type GetInvalid() {
+ return 0;
+ }
+
+ static void Close(handle_type h) {
+ ::UnmapViewOfFile(h);
+ }
+};
+
struct FileHandleTraits : CommonHandleTraits {};
typedef ScopedHandle<CommonHandleTraits> ScopedCommonHandle;
@@ -135,6 +153,8 @@ typedef ScopedHandle<FileHandleTraits> ScopedFileHandle;
typedef ScopedHandle<CryptContextTraits> ScopedCryptContext;
typedef ScopedHandle<FindHandleTraits> ScopedFindHandle;
typedef ScopedHandle<JobHandleTraits> ScopedJobHandle;
+typedef ScopedHandle<FileMappingHandleTraits> ScopedFileMappingHandle;
+typedef ScopedHandle<MappedViewOfFileHandleTraits> ScopedMappedViewOfFileHandle;
namespace llvm {
template <class T>
diff --git a/unittests/Support/Path.cpp b/unittests/Support/Path.cpp
index c791184c83..4982ed0940 100644
--- a/unittests/Support/Path.cpp
+++ b/unittests/Support/Path.cpp
@@ -304,4 +304,32 @@ TEST_F(FileSystemTest, Magic) {
}
}
+TEST_F(FileSystemTest, Canonicalize) {
+ SmallString<128> file_pathname(TestDirectory);
+ path::append(file_pathname, "canonicalize", "a0", "aa1");
+
+ bool existed;
+ ASSERT_NO_ERROR(fs::create_directories(Twine(TestDirectory)
+ + "/canonicalize/a0/aa1", existed));
+
+ {
+ path::append(file_pathname, "file.txt");
+ std::string ErrMsg;
+ raw_fd_ostream file(file_pathname.c_str(), ErrMsg);
+ file << "hello\n";
+ }
+
+ SmallString<0> res;
+ ASSERT_NO_ERROR(fs::canonicalize(Twine(TestDirectory)
+ + "/cAnOnIcAlIzE/A0/aA1/fIlE.TxT", res));
+ // Only check if we actually found the file. As we won't find it on case
+ // sensitive file systems.
+ if (fs::exists(res.str())) {
+ ASSERT_TRUE(res.str().find("canonicalize") != StringRef::npos);
+ ASSERT_TRUE(res.str().find("a0") != StringRef::npos);
+ ASSERT_TRUE(res.str().find("aa1") != StringRef::npos);
+ ASSERT_TRUE(res.str().find("file.txt") != StringRef::npos);
+ }
+}
+
} // anonymous namespace