https://github.com/charles-zablit created 
https://github.com/llvm/llvm-project/pull/172318

This NFC patch improves removes documentation references to methods that have 
been removed in the PseudoTerminal class and adds some `\see` references.

>From a5729ddde9d67364cc7a47e790a3bbc7ff29ab09 Mon Sep 17 00:00:00 2001
From: Charles Zablit <[email protected]>
Date: Mon, 15 Dec 2025 15:33:57 +0000
Subject: [PATCH] [NFC][lldb] fix PseudoTerminal documentation

---
 lldb/include/lldb/Host/PseudoTerminal.h | 20 ++++++++++----------
 1 file changed, 10 insertions(+), 10 deletions(-)

diff --git a/lldb/include/lldb/Host/PseudoTerminal.h 
b/lldb/include/lldb/Host/PseudoTerminal.h
index bd1e2f56241b2..245deec997d80 100644
--- a/lldb/include/lldb/Host/PseudoTerminal.h
+++ b/lldb/include/lldb/Host/PseudoTerminal.h
@@ -27,18 +27,15 @@ class PseudoTerminal {
     invalid_fd = -1 ///< Invalid file descriptor value
   };
 
-  /// Default constructor
-  ///
   /// Constructs this object with invalid primary and secondary file
   /// descriptors.
   PseudoTerminal();
 
-  /// Destructor
+  /// The destructor will close the primary and secondary file
+  /// descriptor/HANDLEs if they are valid and ownership has not been released.
   ///
-  /// The destructor will close the primary and secondary file descriptors if
-  /// they are valid and ownership has not been released using one of: @li
-  /// PseudoTerminal::ReleasePrimaryFileDescriptor() @li
-  /// PseudoTerminal::ReleaseSaveFileDescriptor()
+  /// \see PseudoTerminal::ReleasePrimaryFileDescriptor()
+  /// \see PseudoTerminal::ReleaseSecondaryFileDescriptor()
   ~PseudoTerminal();
 
   /// Close the primary file descriptor if it is valid.
@@ -58,14 +55,17 @@ class PseudoTerminal {
   /// closed as it is no longer needed by the child process.
   ///
   /// This class will close the file descriptors for the primary/secondary when
-  /// the destructor is called. The file handles can be released using either:
-  /// @li PseudoTerminal::ReleasePrimaryFileDescriptor() @li
-  /// PseudoTerminal::ReleaseSaveFileDescriptor()
+  /// the destructor is called. The file handles can be released using one of:
+  /// @li PseudoTerminal::ReleasePrimaryFileDescriptor()
+  /// @li PseudoTerminal::ReleaseSecondaryFileDescriptor()
   ///
   /// \return
   ///     \b Parent process: a child process ID that is greater
   ///         than zero, or an error if the fork fails.
   ///     \b Child process: zero.
+  ///
+  /// \see PseudoTerminal::ReleasePrimaryFileDescriptor()
+  /// \see PseudoTerminal::ReleaseSecondaryFileDescriptor()
   llvm::Expected<lldb::pid_t> Fork();
 
   /// The primary file descriptor accessor.

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

Reply via email to