Sweet. Glad to see that such a large patch had only minor implications. Thanks for the help!
On Wed, Nov 16, 2016 at 11:28 AM Todd Fiala <todd.fi...@gmail.com> wrote: > tfiala accepted this revision. > tfiala added a comment. > This revision is now accepted and ready to land. > > Here is the adjusted patch that fixes the issue I was seeing on > TestTerminal.py. My first set of changes had a lifetime issue where I > needed a const char* that was synthesized on the fly and went away by the > time I needed it. > > LGTM on macOS with this patch applied: > F2590224: D26698_macos_build_fixes_v2.diff < > https://reviews.llvm.org/F2590224> > > > https://reviews.llvm.org/D26698 > > > >
_______________________________________________ lldb-commits mailing list lldb-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
