<div dir="ltr"><br><div class="gmail_extra"><br><br><div class="gmail_quote">On Tue, Jun 4, 2013 at 10:11 PM, Alexander Kornienko <span dir="ltr"><<a href="mailto:alexfh@google.com" target="_blank">alexfh@google.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex"><div dir="ltr"><div class="im">On Tue, Jun 4, 2013 at 10:04 PM, David Blaikie <span dir="ltr"><<a href="mailto:dblaikie@gmail.com" target="_blank">dblaikie@gmail.com</a>></span> wrote:<br>
</div><div class="gmail_extra"><div class="gmail_quote"><div class="im">
<blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex"><div>On Tue, Jun 4, 2013 at 1:01 PM, Jordan Rose <<a href="mailto:jordan_rose@apple.com" target="_blank">jordan_rose@apple.com</a>> wrote:<br>


><br>
>   That's not right. What if I'm running my tool from the command line? Better to just take those out of the compilation database.<br>
<br>
</div>I've a slight tendency to agree - though I'm open to<br>
correction/countersuggestion: the compilation database only really<br>
needs the things that affect compilation. User-features (error limit?<br>
caret diagnostics? color diagnostics?) don't seem relevant to the<br>
database, but perhaps there's a use case I've not thought of. (or that<br>
no one has thought of - which would advocate in favor of leaving them<br>
in "just in case", perhaps)<br></blockquote><div><br></div></div><div>Does it mean that you would better let compilation database filter out flags? I'd say that this is not compilation database's responsibility. And it will be needed in every compilation database implementation.</div>
</div></div></div></blockquote><div><br></div><div>My point of view is that it should not be the compilation database's responsibility : it just can not know what has to be filtered or not for the user --- the filtering should be left to the compilation database user. For the vim case, clang_complete has such a filtering implemented. I did commit <span style="color:rgb(31,28,27);font-family:'Sans Serif';font-size:11px">r182878 --- </span>which introduced this change --- and it can be reverted : arguably, we can consider that either ninja or clang need to be fixed for proper color handling / terminal capabilities detection.</div>
<div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex">_______________________________________________<br>

cfe-commits mailing list<br>
<a href="mailto:cfe-commits@cs.uiuc.edu">cfe-commits@cs.uiuc.edu</a><br>
<a href="http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits" target="_blank">http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits</a><br>
<br></blockquote></div><br></div></div>