<div dir="ltr">The FileSpec is not even used for anything is it?  If you want to use the RegularExpression, then use llvm's regular expression.  Honestly LLDB's regular expression should be deprecated anyway, there's no reason to have an LLDB regex and an LLVM regex when just one will suffice</div><br><div class="gmail_quote"><div dir="ltr">On Thu, Sep 10, 2015 at 5:04 PM Dawn Perchik <<a href="mailto:dawn%2Bllvm@burble.org">dawn+llvm@burble.org</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">dawn added a comment.<br>
<br>
In <a href="http://reviews.llvm.org/D12115#243093" rel="noreferrer" target="_blank">http://reviews.llvm.org/D12115#243093</a>, @abidh wrote:<br>
<br>
> We dont use the lldb_private stuff inside lldb-mi. Please see the discussion in the following thread.<br>
>  <a href="http://lists.llvm.org/pipermail/lldb-dev/2015-March/007047.html" rel="noreferrer" target="_blank">http://lists.llvm.org/pipermail/lldb-dev/2015-March/007047.html</a><br>
<br>
<br>
Thank you for your review and for the link.  I agree in general and have avoided lldb_private myself, but this means duplicated code which is arguably worse.  I think it's time to make lldb-mi a proper part of lldb to avoid reinventing the wheel, or adding SB APIs that make no sense outside of lldb-mi.  But I am sympathetic to both arguments so will revise.<br>
<br>
<br>
Repository:<br>
  rL LLVM<br>
<br>
<a href="http://reviews.llvm.org/D12115" rel="noreferrer" target="_blank">http://reviews.llvm.org/D12115</a><br>
<br>
<br>
<br>
</blockquote></div>