Ahh, no need, if that's the case it's fine.  I'll look at the patch in a bit (on mobile right now)<br><div class="gmail_quote">On Wed, Feb 4, 2015 at 9:19 AM Hafiz Abid Qadeer <<a href="mailto:abidh.haq@gmail.com">abidh.haq@gmail.com</a>> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">On Linux, eclipse send the normal path like<br>
37-break-insert -f /home/abidh/work/test.c:28<br>
<br>
I agree with that check about ePathSyntaxWindows. We dont want to put penalty on other platforms. But looking at Normalize function, it seems that std::replace only gets called when we have syntax == ePathSyntaxWindows. It returns early for ePathSyntaxPosix. But I can put an explicit check if required.<br>
<br>
<br>
<a href="http://reviews.llvm.org/D7379" target="_blank">http://reviews.llvm.org/D7379</a><br>
<br>
EMAIL PREFERENCES<br>
  <a href="http://reviews.llvm.org/settings/panel/emailpreferences/" target="_blank">http://reviews.llvm.org/<u></u>settings/panel/<u></u>emailpreferences/</a><br>
<br>
<br>
</blockquote></div>