[Lldb-commits] [PATCH] D16149: Delete char const** swig typemaps

Zachary Turner via lldb-commits lldb-commits at lists.llvm.org
Wed Jan 13 13:13:45 PST 2016


zturner added a comment.

I just moved tid to the bottom because it was in the middle of the in, out, and typecheck versions of the char** typemap.  So I wanted to group them all together.

Anyway, yea this didn't seem too controversial to me but I figured I'd throw it up here for review in case you knew of some quirk of swig with regards to typematching and constness.  But yea, I tested it and the generated code is semantically identical.

If anything breaks it will probably be due to a swig version difference, so let me know after I commit since I'm using 3.x


http://reviews.llvm.org/D16149





More information about the lldb-commits mailing list