mgorny added inline comments.
================ Comment at: lldb/source/Host/posix/ConnectionFileDescriptorPosix.cpp:772-774 + llvm::Error error = llvm::Error::success(); + m_io_sp = std::make_shared<SerialPort>(fd, File::eOpenOptionReadWrite, + serial_options.get(), true, error); ---------------- labath wrote: > It would be better to hide this by making the constructor private and have a > static factory function returning `Expected<unique_ptr<SerialPort>>`. Bonus > points if you can move all the fallible operations into the factory function > so that the (now private) constructor does not need the Error argument at all I know but I couldn't make this work — the compiler just won't find a way to convert the new `SerialPort` instance into `Expected<SerialPort>`. I suspect I would have to add move ctors and assignment operators all the way up the class hierarchy. CHANGES SINCE LAST ACTION https://reviews.llvm.org/D111355/new/ https://reviews.llvm.org/D111355 _______________________________________________ lldb-commits mailing list lldb-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
