Er, my suggestion was to make these *static* variables, yes, even though they are in a header file, and then to document thoroughly that this header file is *not* for use within a library, but for use directly within the file defining main.<div>
<br></div><div>The current patch is different from that, can you describe how? In general the comments weren't clear enough for me to be confident in how you expect this file to be used...</div><div class="gmail_extra">
<br><br><div class="gmail_quote">On Fri, Aug 17, 2012 at 2:26 AM, 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">
+using namespace llvm;<br>
<br>
Remove from header.<br>
<br>
Appart from that, lgtm, minus Chandler's approval of the general idea.<br>
<div class="HOEnZb"><div class="h5"><br>
On Thu, Aug 16, 2012 at 7:17 PM, Alexander Kornienko <<a href="mailto:alexfh@google.com">alexfh@google.com</a>> wrote:<br>
> On Thu, Aug 16, 2012 at 7:04 PM, Manuel Klimek <<a href="mailto:klimek@google.com">klimek@google.com</a>> wrote:<br>
>><br>
>> On Thu, Aug 16, 2012 at 5:45 PM, Alexander Kornienko <<a href="mailto:alexfh@google.com">alexfh@google.com</a>><br>
>> wrote:<br>
>> > Now it's a bit uglier, but doesn't use CommandLine Library in an<br>
>> > unsupported<br>
>> > way.<br>
>> > Chandler, please take a look if it seems better to you.<br>
>><br>
>> 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>
><br>
> A new patch is attached.<br>
><br>
>><br>
>><br>
>> > BTW, we still have a number of alien options defined in some llvm<br>
>> > libraries<br>
>> > we link with. Should we try to avoid that by de-globalizing cl library<br>
>> > or is<br>
>> > someone working on its replacement/refactoring now?<br>
><br>
><br>
> --<br>
> Best regards,<br>
> Alexander Kornienko<br>
><br>
</div></div></blockquote></div><br></div>