[Lldb-commits] [PATCH] D68188: allow arbitrary python streams to be converted to SBFile
Lawrence D'Anna via Phabricator via lldb-commits
lldb-commits at lists.llvm.org
Fri Oct 4 18:25:21 PDT 2019
lawrence_danna added a comment.
In D68188#1695069 <https://reviews.llvm.org/D68188#1695069>, @labath wrote:
> I agree about the separate patch stuff, but it seems to be that this should be done before this one. After all, all (most?) of the existing code has already been DataObject-ized and this patch is the thing that's deviating from that practice. I don't think you should rewrite all of the existing code -- hopefully we can find a way to fit this in there.
Yea... I tried updating all the PythonDataObjects code to use Errors and Expected<> and it wound up being just way too big of a patch.
I've updated this patch to add a giant warning at the top of PythonDataObjects.h about the dangers, as well adding just enough Expected<> support to do what I need for this patch.
How's it look now?
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
More information about the lldb-commits