Re: [PATCH v1] config.c: fix msvc compile error

From: Jeff Hostetler <git@jeffhostetler.com>
Date: Jul 25, 2018, 12:51 UTC
Message-ID: <ca10e6f5-91be-e2a9-19a8-37b67217d134@jeffhostetler.com>
In-Reply-To: <20180724195637.GA97109@syl.attlocal.net>

On 7/24/2018 3:56 PM, Taylor Blau wrote:

Show 19 quoted lines

> On Tue, Jul 24, 2018 at 03:30:10PM +0000, git@jeffhostetler.com wrote:
>> From: Jeff Hostetler <jeffhost@microsoft.com>
>>
>> In commit fb0dc3bac135e9f6243bd6d293e8c9293c73b9cd code was added
>> to builtin/config.c to define a new function and a forward declaration
>> for an array of unknown size.  This causes a compile error under MSVC.
> 
> Thanks for spending time fixing that. fb0dc3bac1 (builtin/config.c:
> support `--type=<type>` as preferred alias for `--<type>`, 2018-04-18)
> is from me, so I owe you an extra thanks for patching up mistakes :-).
> 
> As others have noted in this thread, another patch was sent into similar
> effect, which has already been queued, and I agree that we should prefer
> that, since it's further along.
> 
> 
> Thanks,
> Taylor
>

Yes, the other version is further along. Let's take it. Jeff