[Patch] Option parsing: allow option aliases in option groups

[Hans Wennborg]

Thanks! Committed in r186838.

On Mon, Jul 22, 2013 at 7:38 AM, Reid Kleckner <rnk at google.com> wrote:
> LGTM
>
>
> On Fri, Jul 19, 2013 at 8:34 PM, Hans Wennborg <hans at chromium.org> wrote:
>>
>> Hi,
>>
>> This patch makes the option parsing code allow for having option
>> aliases in option groups.
>>
>> Aliases in groups were previously disallowed by an assert. As far as I
>> know, there is no technical reason for this, and I would like to allow
>> them, to be able to put cl.exe compatible options in their own group,
>> but have some of them (like "/c" for example) be aliases.
>>
>> Please take a look.
>>
>> Thanks,
>> Hans
>
>