[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