<div dir="ltr">Yep, done in r209760.</div><div class="gmail_extra"><br><br><div class="gmail_quote">On Wed, May 28, 2014 at 8:06 AM, Zachary Turner <span dir="ltr"><<a href="mailto:zturner@google.com" target="_blank">zturner@google.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Thanks!  Could I request someone to submit the patch for me?<br>
<div class="HOEnZb"><div class="h5"><br>
On Wed, May 28, 2014 at 7:29 AM, Aaron Ballman <<a href="mailto:aaron@aaronballman.com">aaron@aaronballman.com</a>> wrote:<br>
> On Tue, May 27, 2014 at 4:00 PM, Zachary Turner <<a href="mailto:zturner@google.com">zturner@google.com</a>> wrote:<br>
>> I've attached the updated patch.  This is my first patch, so I'm not<br>
>> sure what the procedure is for making changes, but this updated diff<br>
>> is based on HEAD rather than on my previous diff.  I think this is the<br>
>> right thing to do, but let me know if I'm wrong.<br>
>><br>
>> I put the #error outside of the #if _MSVC check.  The reason for this<br>
>> is that, even though the non-MSVC branch was using #if HAVE_STDINT_H<br>
>> previously, it was not doing anything in case the header didn't exist.<br>
>>  In other words, for any other platform where stdint was not present,<br>
>> it was already broken.<br>
><br>
> LGTM, at least for the CMake side of things.<br>
><br>
> Thanks!<br>
><br>
> ~Aaron<br>
><br>
>><br>
>> On Tue, May 27, 2014 at 12:20 PM, Zachary Turner <<a href="mailto:zturner@google.com">zturner@google.com</a>> wrote:<br>
>>> This was actually my original approach until I searched around online.  One<br>
>>> example of a report of a user not being able to find stdint.h on VS2012 is<br>
>>> here: <a href="https://code.google.com/p/msinttypes/issues/detail?id=12" target="_blank">https://code.google.com/p/msinttypes/issues/detail?id=12</a><br>
>>><br>
>>> That being said, i did in fact do a vanilla install of VS2012 and confirmed<br>
>>> that the file was there.  So I really don't know how to explain that<br>
>>> person's experience.  Maybe a pre-release version of VS2012?  You can find<br>
>>> more reports of people not having stdint.h by doing a quick google search<br>
>>> for "visual studio 2012 stdint.h".<br>
>>><br>
>>> I would also prefer to remove all of this, so if everyone is still ok with<br>
>>> this, I will change the patch accordingly.<br>
>>><br>
>>><br>
>>> On Tue, May 27, 2014 at 11:33 AM, Aaron Ballman <<a href="mailto:aaron@aaronballman.com">aaron@aaronballman.com</a>><br>
>>> wrote:<br>
>>>><br>
>>>> On Tue, May 27, 2014 at 2:30 PM, Reid Kleckner <<a href="mailto:rnk@google.com">rnk@google.com</a>> wrote:<br>
>>>> > +Nakamura, he knows more about the various supported Windows configs.<br>
>>>> ><br>
>>>> > Thanks!  At one point I thought I could delete DataTypes.h completely in<br>
>>>> > favor of stdint.h, but I gave up because inttypes.h makes things<br>
>>>> > complicated.<br>
>>>> ><br>
>>>> > +#if defined(_WIN64)<br>
>>>> > +typedef signed __int64 ssize_t;<br>
>>>> > +#else<br>
>>>> > +typedef signed int ssize_t;<br>
>>>> > +#endif /* _WIN64 */<br>
>>>> > +<br>
>>>> > +/* Ideally, we would like to assume that stdint.h always exists on<br>
>>>> > +   every supported Visual Studio version.  Indeed, the header does get<br>
>>>> > +   installed on a vanilla VS 2012 installation.  But a search online<br>
>>>> > +   shows that some people seem to have a version of VS2012 that does<br>
>>>> > +   not have this header, so without a better understanding of why this<br>
>>>> > +   is it's better to leave this check in. */<br>
>>>> > +#ifndef HAVE_STDINT_H<br>
>>>> ><br>
>>>> > Really?  I thought it was available since VS 2010.  Do you have a link<br>
>>>> > to<br>
>>>> > these results?  I'd like to nuke this, replace it with '#error "MSVC<br>
>>>> > must<br>
>>>> > provide stdint.h"', commit it, and see if it sticks.<br>
>>>><br>
>>>> stdint.h is included with MSVC 2010 and later. I concur with the<br>
>>>> #error approach.<br>
>>>><br>
>>>> ~Aaron<br>
>>><br>
>>><br>
</div></div></blockquote></div><br></div>