zturner added inline comments.

================
Comment at: scripts/Python/python-typemaps.swig:532
@@ -531,3 +524,1 @@
-         file.Clear();
-    }
 }
----------------
zturner wrote:
> fjricci wrote:
> > The problem is that here, we save the `FILE*` (`$1`) and let the `File` 
> > (`file`) go out of scope. So the `File` gets destructed (but it's after 
> > calling `file.Clear()`, so the close doesn't happen). But we still hold 
> > onto the `FILE*`, and we pass it, as `$1`, to the API call (not seen here, 
> > but it comes right after this block in the generated swig cpp code).
> I think SWIG explicitly has a mechanism to handle this.  Let me find it.
It's the `freearg` [[ http://www.swig.org/Doc1.3/Typemaps.html#Typemaps_nn33 | 
typemap ]].

Basically you can put a rule that just calls `fclose()` on the argument, and it 
will generate this code after it's called the API.

Would this work?


http://reviews.llvm.org/D18459



_______________________________________________
lldb-commits mailing list
lldb-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits

Reply via email to