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



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

Reply via email to