<div class="gmail_quote">On Thu, Aug 16, 2012 at 7:04 PM, Manuel Klimek <span dir="ltr"><<a href="mailto:klimek@google.com" target="_blank">klimek@google.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<div class="im">On Thu, Aug 16, 2012 at 5:45 PM, Alexander Kornienko <<a href="mailto:alexfh@google.com">alexfh@google.com</a>> wrote:<br>
> Now it's a bit uglier, but doesn't use CommandLine Library in an unsupported<br>
> way.<br>
> Chandler, please take a look if it seems better to you.<br>
<br>
</div>After a short discussion off-list we came to the conclusion that<br>
tooling::CommonOptionsParser is a better name for the class, and<br>
especially makes the responsibilities clearer...<br></blockquote><div>A new patch is attached.</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<div class="HOEnZb"><div class="h5"><br>
> BTW, we still have a number of alien options defined in some llvm libraries<br>
> we link with. Should we try to avoid that by de-globalizing cl library or is<br>
> someone working on its replacement/refactoring now?<br>
</div></div></blockquote></div><br>-- <br><div>Best regards,</div><div>Alexander Kornienko</div><br>