[Lldb-commits] [PATCH] D50161: Add raw_ostream wrapper to the Stream class
Pavel Labath via Phabricator via lldb-commits
lldb-commits at lists.llvm.org
Thu Aug 2 02:50:43 PDT 2018
labath added a comment.
Wouldn't it be even better to actually expose the llvm class via some accessor or something? This way we could slowly migrate existing code by changing it to write to `stream.accessor()` instead of `stream` ? (I am not saying to do that now, but it opens up possibilities for the future.)
I also found it hard to deduce who is forwarding to whom from the member names. Maybe rename `m_forward_to` to `m_target` or something like that?
More information about the lldb-commits