<div dir="ltr">LGTM</div><div class="gmail_extra"><br><br><div class="gmail_quote">On Fri, Jul 19, 2013 at 8:34 PM, Hans Wennborg <span dir="ltr"><<a href="mailto:hans@chromium.org" target="_blank">hans@chromium.org</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Hi,<br>
<br>
This patch makes the option parsing code allow for having option<br>
aliases in option groups.<br>
<br>
Aliases in groups were previously disallowed by an assert. As far as I<br>
know, there is no technical reason for this, and I would like to allow<br>
them, to be able to put cl.exe compatible options in their own group,<br>
but have some of them (like "/c" for example) be aliases.<br>
<br>
Please take a look.<br>
<br>
Thanks,<br>
Hans<br>
</blockquote></div><br></div>