[Lldb-commits] [PATCH] D38829: Python: SetOutputFileHandle doesn't work with IOBase

[Pavel Labath via Phabricator via lldb-commits]

labath added a comment.

In https://reviews.llvm.org/D38829#914300, @zturner wrote:

> Ok, I wasn't aware of the libedit problem.
>
> I still don't like this approach, because it causes the design of the `File` class to be influenced by a limitation of a 3rd party library.
>
> What about adding a method to `IOObject` called `FILE *GetFileStream()`.  Then the method I proposed earlier would just involve `PythonFileIo` implementing this in the proper way.  Then we can pass `SBFile` or `lldb_private::File` through all layers of the codebase, and once we're in libedit we just call `file.GetFileStream()` and pass it to libedit?


That's how I'd do it.

(*) libedit needs both a FILE* and a raw file descriptor in this case (I think it's calling select, and some terminal functions).


Repository:
  rL LLVM

https://reviews.llvm.org/D38829