diff options
author | Lawrence D'Anna <lawrence_danna@apple.com> | 2019-10-03 04:31:46 +0000 |
---|---|---|
committer | Lawrence D'Anna <lawrence_danna@apple.com> | 2019-10-03 04:31:46 +0000 |
commit | f913fd6eb0c228041b77a645d76993760d3b1421 (patch) | |
tree | 9d33e1b8a85f1fbccf83cc3f59efeed7aa4511c3 /lldb/source/Host/posix/ConnectionFileDescriptorPosix.cpp | |
parent | 96898eb6a935533aaebbfbd085150fbf705c0ffc (diff) | |
download | bcm5719-llvm-f913fd6eb0c228041b77a645d76993760d3b1421.tar.gz bcm5719-llvm-f913fd6eb0c228041b77a645d76993760d3b1421.zip |
factor out an abstract base class for File
Summary:
This patch factors out File as an abstract base
class and moves most of its actual functionality into
a subclass called NativeFile. In the next patch,
I'm going to be adding subclasses of File that
don't necessarily have any connection to actual OS files,
so they will not inherit from NativeFile.
This patch was split out as a prerequisite for
https://reviews.llvm.org/D68188
Reviewers: JDevlieghere, jasonmolenda, labath
Reviewed By: labath
Subscribers: lldb-commits
Tags: #lldb
Differential Revision: https://reviews.llvm.org/D68317
llvm-svn: 373564
Diffstat (limited to 'lldb/source/Host/posix/ConnectionFileDescriptorPosix.cpp')
-rw-r--r-- | lldb/source/Host/posix/ConnectionFileDescriptorPosix.cpp | 12 |
1 files changed, 6 insertions, 6 deletions
diff --git a/lldb/source/Host/posix/ConnectionFileDescriptorPosix.cpp b/lldb/source/Host/posix/ConnectionFileDescriptorPosix.cpp index f9759ec3caf..325d854921e 100644 --- a/lldb/source/Host/posix/ConnectionFileDescriptorPosix.cpp +++ b/lldb/source/Host/posix/ConnectionFileDescriptorPosix.cpp @@ -86,8 +86,8 @@ ConnectionFileDescriptor::ConnectionFileDescriptor(bool child_processes_inherit) ConnectionFileDescriptor::ConnectionFileDescriptor(int fd, bool owns_fd) : Connection(), m_pipe(), m_mutex(), m_shutting_down(false), m_waiting_for_accept(false), m_child_processes_inherit(false) { - m_write_sp = std::make_shared<File>(fd, File::eOpenOptionWrite, owns_fd); - m_read_sp = std::make_shared<File>(fd, File::eOpenOptionRead, false); + m_write_sp = std::make_shared<NativeFile>(fd, File::eOpenOptionWrite, owns_fd); + m_read_sp = std::make_shared<NativeFile>(fd, File::eOpenOptionRead, false); Log *log(lldb_private::GetLogIfAnyCategoriesSet(LIBLLDB_LOG_CONNECTION | LIBLLDB_LOG_OBJECT)); @@ -219,9 +219,9 @@ ConnectionStatus ConnectionFileDescriptor::Connect(llvm::StringRef path, m_write_sp = m_read_sp; } else { m_read_sp = - std::make_shared<File>(fd, File::eOpenOptionRead, false); + std::make_shared<NativeFile>(fd, File::eOpenOptionRead, false); m_write_sp = - std::make_shared<File>(fd, File::eOpenOptionWrite, false); + std::make_shared<NativeFile>(fd, File::eOpenOptionWrite, false); } m_uri = *addr; return eConnectionStatusSuccess; @@ -270,8 +270,8 @@ ConnectionStatus ConnectionFileDescriptor::Connect(llvm::StringRef path, ::fcntl(fd, F_SETFL, flags); } } - m_read_sp = std::make_shared<File>(fd, File::eOpenOptionRead, true); - m_write_sp = std::make_shared<File>(fd, File::eOpenOptionWrite, false); + m_read_sp = std::make_shared<NativeFile>(fd, File::eOpenOptionRead, true); + m_write_sp = std::make_shared<NativeFile>(fd, File::eOpenOptionWrite, false); return eConnectionStatusSuccess; } #endif |