<div dir="ltr">I'm not sure if this is a step toward tools acting as CXX replacements but I renamed the class to something a little more describe of what it does as opposed to what it's used for.</div><div class="gmail_extra">

<br><br><div class="gmail_quote">On Fri, Oct 11, 2013 at 3:13 AM, Kim Gräsman <span dir="ltr"><<a href="mailto:kim.grasman@gmail.com" target="_blank">kim.grasman@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">

Hi Edwin,<br>
<div class="im"><br>
On Fri, Oct 11, 2013 at 12:34 AM, Edwin Vane <<a href="mailto:edwin.vane@intel.com">edwin.vane@intel.com</a>> wrote:<br>
><br>
>   The problem is the FixedCompilationDatabase mangles the arguments you pass. FixedCompilationDatabase as a result has some preconditions: the name of the compiler should not be present nor should be the filename to operate on. Check CompilationDatabase.h for a better description. As stated in the patch summary we need a simpler compilation database that simply accepts a full list of args and then provides those same args back upon request.<br>


<br>
</div>This sounds nice, but the name feels a little funny... Isn't it just a<br>
LiteralCompilationDatabase? The subprocess forwarding is just one use<br>
case, unless I'm missing something. I'm still interested in allowing<br>
tools to act as CXX replacements in build systems, for example, and it<br>
feels like this is as step on the way there?<br>
<span class="HOEnZb"><font color="#888888"><br>
- K<br>
</font></span><div class="HOEnZb"><div class="h5"><br>
_______________________________________________<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>
</div></div></blockquote></div><br><br clear="all"><div><br></div>-- <br>Edwin V
</div>