<div dir="ltr">I don't think that's correct.  I think directory should be tmp and filename should be null in that case.<div><br></div><div>It's interesting that this happens when resolve == false, I would have thought most differences to arise when resolve == true.</div><div><br></div><div>It's too bad we didn't have test/functionalities/paths back then.  It's hard to know what's breaking and what we need to verify each time we make a change.<br><div><br></div></div><div>Whatever the decision is, I think we should prioritize building up a comprehensive set of tests that hit all the edge cases.  I can volunteer to go in and add a bunch of windows specific tests and edge cases, if someone else will do the linux ones.</div><div><br></div><div>This is at least the 4th time in recent memory that FileSpec has bitten us, so it's probably woth setting aside some time and building up a bunch of test cases from scratch.</div></div><br><div class="gmail_quote">On Wed, Mar 11, 2015 at 2:35 PM Greg Clayton <<a href="mailto:gclayton@apple.com">gclayton@apple.com</a>> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">FileSpec was changed a while back and currently if we make call like:<br>
<br>
FileSpec tmp("/tmp", false);<br>
<br>
will result in a m_filename that contains "." and a m_directory that contains "/tmp". Is this expected?<br>
<br>
If so we should to change:<br>
<br>
void<br>
FileSpec::AppendPathComponent (const char *new_path);<br>
<br>
to not make the path be "/tmp/./<new_path>"...<br>
<br>
Greg<br>
______________________________<u></u>_________________<br>
lldb-dev mailing list<br>
<a href="mailto:lldb-dev@cs.uiuc.edu" target="_blank">lldb-dev@cs.uiuc.edu</a><br>
<a href="http://lists.cs.uiuc.edu/mailman/listinfo/lldb-dev" target="_blank">http://lists.cs.uiuc.edu/<u></u>mailman/listinfo/lldb-dev</a><br>
</blockquote></div>