[Lldb-commits] [PATCH] D68188: allow arbitrary python streams to be converted to SBFile

Pavel Labath via Phabricator via lldb-commits lldb-commits at lists.llvm.org
Wed Oct 2 07:25:05 PDT 2019

labath added a comment.

In D68188#1691337 <https://reviews.llvm.org/D68188#1691337>, @lawrence_danna wrote:

> In D68188#1691129 <https://reviews.llvm.org/D68188#1691129>, @labath wrote:
> > Then depending on what you need, you create either a NativeFile, OwningPythonFile<NativeFile>, or a Text/BinaryPythonFile. How does that sound?
> Next step is add type maps to convert these things back to python.  Without the CRTP it can’t just check for a single OwnedPythonfile base class — it’ll have to check for all three.  Is that worse than using CRTP?

Hmm... hard to say without knowing more details, but I would say that being worse than CRTP is quite hard. :) Also, Text and Binary python file will have OwningPythonFile<File> as their base, so you can check for those together. I'd say we go with this first, and then revisit if needed.

  rG LLVM Github Monorepo



More information about the lldb-commits mailing list