[Lldb-commits] [PATCH] D18459: Fix FILE * leak in Python API

Francis Ricci via lldb-commits lldb-commits at lists.llvm.org
Fri Mar 25 09:45:20 PDT 2016


fjricci added a comment.

I think that would be sufficient to solve the problem.

I could use a `%rename` to rename the `SetImmediateOutputFile (FILE *fh)` call to a private function in `%extend`, which will then call `SetImmediateOutputFile (FILE *fh, transfer_ownership=true)` instead.

Does that seem reasonable? I don't see any `%rename` calls in the existing lldb source, which is my primary hesitation.


http://reviews.llvm.org/D18459





More information about the lldb-commits mailing list