[Lldb-commits] [PATCH] Add SBArgs to the public interface

Enrico Granata egranata at apple.com
Wed Mar 11 14:53:56 PDT 2015


I see

Looking at your patch, I see that it uses std::string in the SB layer
That’s a big no-no, so please rework your code to use SBStream or const char* + a size_t (and SWIG should do the right thing for Python)

> On Mar 11, 2015, at 2:43 PM, Zachary Turner <zturner at google.com> wrote:
> 
> Mostly added you for the SWIG stuff, because I'm doing something kind of "new" here and I had to figure it out with some hacking.
> 
> +Greg for the overall idea.
> 
> 
> http://reviews.llvm.org/D8265
> 
> EMAIL PREFERENCES
>  http://reviews.llvm.org/settings/panel/emailpreferences/
> 
> 
> 
> _______________________________________________
> lldb-commits mailing list
> lldb-commits at cs.uiuc.edu
> http://lists.cs.uiuc.edu/mailman/listinfo/lldb-commits





More information about the lldb-commits mailing list