================
@@ -20,11 +21,18 @@ namespace lldb_dap {
 ///
 /// The file is destroyed when the destructor is invoked.
 struct FifoFile {
-  FifoFile(llvm::StringRef path);
+  FifoFile(llvm::StringRef path, lldb::pipe_t pipe = LLDB_INVALID_PIPE);
 
   ~FifoFile();
----------------
ashgti wrote:

I think we should mark this as a move only type to ensure we don't end up with 
an implicit copy.

Can we make this into a class and remove the copy constructor / `operator=`? 
(e.g. like 
https://github.com/llvm/llvm-project/blob/0d23e3f71f56721e9c050bda63d10c7fd44de117/lldb/tools/lldb-dap/DAP.h#L204-L208)

https://github.com/llvm/llvm-project/pull/174635
_______________________________________________
lldb-commits mailing list
[email protected]
https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits

Reply via email to