<div style="white-space:pre-wrap">Assert probably </div><br><div class="gmail_quote"><div dir="ltr">On Mon, Sep 5, 2016 at 4:04 AM Pavel Labath <<a href="mailto:labath@google.com">labath@google.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">labath added a subscriber: labath.<br>
labath added a comment.<br>
<br>
<br>
<br>
<br>
================<br>
Comment at: include/lldb/Host/FileSpec.h:712<br>
@@ +711,3 @@<br>
+             PathSyntax syntax = ePathSyntaxHostNative,<br>
+             llvm::Triple *triple = nullptr);<br>
+<br>
----------------<br>
I don't think the default-null parameter here is a good idea. This means some of your file-specs will be case-sensitive and some will not (depending on whether the the person who created the FileSpec remembered to pass in the triple).<br>
<br>
If it were up to me, I wouldn't even make case-sensitivity a property of the file spec. I think it should be a property of the comparison between them. Otherwise, what are you going to do when someone asks you to compare a case-insensitive "/foo/bar" with a case-sensitive "/Foo/Bar" ?<br>
<br>
<br>
<a href="https://reviews.llvm.org/D20041" rel="noreferrer" target="_blank">https://reviews.llvm.org/D20041</a><br>
<br>
<br>
<br>
</blockquote></div>