[RFC][PATCH 1/5] [PR gdb/33469] ui_out: add optional async param to redirect()

["Dennis Lambe Jr." <malsyned@malsyned.net>]

This adds a new overload of the protected do_redirect(), rather than
adding an extra optional parameter, so that it can provide a default
implementation of the two-parameter form without disturbing
implementations that don't care about this distinction (which is all but
one of them).

Most uses of redirect(), such as in buffered_streams, assume that this
sort of state won't change during the redirect, so the default for the
one-parameter overload is to forward to the two-parameter form with
async = false.
---
 gdb/ui-out.h | 5 ++++-
 1 file changed, 4 insertions(+), 1 deletion(-)

diff --git a/gdb/ui-out.h b/gdb/ui-out.h
index 1796e9c9eb3..f7d6be2b026 100644
--- a/gdb/ui-out.h
+++ b/gdb/ui-out.h
@@ -263,7 +263,8 @@ class ui_out
   void flush () { do_flush (); }
 
   /* Redirect the output of a ui_out object temporarily.  */
-  void redirect (ui_file *outstream) { do_redirect (outstream); }
+  void redirect (ui_file *outstream, bool async = false)
+  { do_redirect (outstream, async); }
 
   ui_out_flags test_flags (ui_out_flags mask)
   { return m_flags & mask; }
@@ -368,6 +369,8 @@ class ui_out
   virtual void do_wrap_hint (int indent) = 0;
   virtual void do_flush () = 0;
   virtual void do_redirect (struct ui_file *outstream) = 0;
+  virtual void do_redirect (struct ui_file *outstream, bool async)
+  { do_redirect (outstream); }
 
   virtual void do_progress_start () = 0;
   virtual void do_progress_notify (const std::string &, const char *,
-- 
2.51.0