<div dir="ltr">Sweet.  Glad to see that such a large patch had only minor implications.  Thanks for the help!</div><br><div class="gmail_quote"><div dir="ltr">On Wed, Nov 16, 2016 at 11:28 AM Todd Fiala <<a href="mailto:todd.fiala@gmail.com">todd.fiala@gmail.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">tfiala accepted this revision.<br class="gmail_msg">
tfiala added a comment.<br class="gmail_msg">
This revision is now accepted and ready to land.<br class="gmail_msg">
<br class="gmail_msg">
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.<br class="gmail_msg">
<br class="gmail_msg">
LGTM on macOS with this patch applied:<br class="gmail_msg">
F2590224: D26698_macos_build_fixes_v2.diff <<a href="https://reviews.llvm.org/F2590224" rel="noreferrer" class="gmail_msg" target="_blank">https://reviews.llvm.org/F2590224</a>><br class="gmail_msg">
<br class="gmail_msg">
<br class="gmail_msg">
<a href="https://reviews.llvm.org/D26698" rel="noreferrer" class="gmail_msg" target="_blank">https://reviews.llvm.org/D26698</a><br class="gmail_msg">
<br class="gmail_msg">
<br class="gmail_msg">
<br class="gmail_msg">
</blockquote></div>