labath added a comment.

In D146590#4220388 <https://reviews.llvm.org/D146590#4220388>, @jgorbe wrote:

> I believe (but I don't have any experience with SWIG typemaps so this is an 
> educated guess) that the call to `free` in the error path comes from the 
> `%typemap(freearg)` immediately after that one. So if freearg already takes 
> care of it, the error handling logic in `%typemap(in)` should just call 
> `SWIG_fail`. Does that sound correct?

Makes sense to me.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D146590/new/

https://reviews.llvm.org/D146590

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

Reply via email to