diff options
| author | Lawrence D'Anna <lawrence_danna@apple.com> | 2019-10-14 20:15:34 +0000 |
|---|---|---|
| committer | Lawrence D'Anna <lawrence_danna@apple.com> | 2019-10-14 20:15:34 +0000 |
| commit | 62c9fe4273e8f2a0f3f0f4c86de3a90668532354 (patch) | |
| tree | 50c6f7a76ff044c13465b9ad7583558d6c275e1d /lldb/source/Host/common/FileSystem.cpp | |
| parent | 322f12afc3673fc868899857b069ce59084dba05 (diff) | |
| download | llvm-62c9fe4273e8f2a0f3f0f4c86de3a90668532354.zip llvm-62c9fe4273e8f2a0f3f0f4c86de3a90668532354.tar.gz llvm-62c9fe4273e8f2a0f3f0f4c86de3a90668532354.tar.bz2 | |
uint32_t options -> File::OpenOptions options
Summary:
This patch re-types everywhere that passes a File::OpenOptions
as a uint32_t so it actually uses File::OpenOptions.
It also converts some OpenOptions related functions that fail
by returning 0 or NULL into llvm::Expected
split off from https://reviews.llvm.org/D68737
Reviewers: JDevlieghere, jasonmolenda, labath
Reviewed By: labath
Subscribers: lldb-commits
Tags: #lldb
Differential Revision: https://reviews.llvm.org/D68853
llvm-svn: 374817
Diffstat (limited to 'lldb/source/Host/common/FileSystem.cpp')
| -rw-r--r-- | lldb/source/Host/common/FileSystem.cpp | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/lldb/source/Host/common/FileSystem.cpp b/lldb/source/Host/common/FileSystem.cpp index ba581ae..2db5bff 100644 --- a/lldb/source/Host/common/FileSystem.cpp +++ b/lldb/source/Host/common/FileSystem.cpp @@ -415,7 +415,8 @@ static mode_t GetOpenMode(uint32_t permissions) { return mode; } -Expected<FileUP> FileSystem::Open(const FileSpec &file_spec, uint32_t options, +Expected<FileUP> FileSystem::Open(const FileSpec &file_spec, + File::OpenOptions options, uint32_t permissions, bool should_close_fd) { if (m_collector) m_collector->addFile(file_spec.GetPath()); |