-
Notifications
You must be signed in to change notification settings - Fork 2.1k
64-Bit Windows Fixes #93
Conversation
|
We found a Contributor License Agreement for you (the sender of this pull request), but were unable to find agreements for the commit author(s). If you authored these, maybe you used a different email address in the git commits than was used to sign the CLA (login here to double check)? If these were authored by someone else, then they will need to sign a CLA as well, and confirm that they're okay with these being contributed to Google. |
|
Hi, This PR should be cla:yes. The commit CLAbot is saying "no" for is by an author that accepted the CLA and has just been merged into glog mainline (#123). |
|
We found a Contributor License Agreement for you (the sender of this pull request) and all commit authors, but as best as we can tell these commits were authored by someone else. If that's the case, please add them to this pull request and have them confirm that they're okay with these commits being contributed to Google. If we're mistaken and you did author these commits, just reply here to confirm. |
|
Any news on this PR? Doesn't appear to have been review. 😕 |
|
@artemdinaburg Could you please remove the commits not authored by you and squash the remaining ones? As far as I can tell, the last commit in the PR has been already merged. Also, please update your email address in your commit such that the bot can detect that you already signed the CLA. |
|
Sorry seems I caused some confusion with my comment. I didn't open this PR or add commits to it. Just was eager to see it reviewed. |
|
@jakirkham My bad. I actually wanted to address @artemdinaburg. Thanks for pointing it out. |
|
No worries. Thanks for taking a look. 🙂 |
|
Changes merged with #331 please close this PR |
This makes unit tests pass on 64-bit Windows by comparing
%pformat strings via a special token. On 64-bit Windows,%pis always zero-padded to 16 characters. This change should be helpful in general, since%pis implementation defined.Also
#definesome constants in windows/port.h to their underscore prefixed versions, as the non-underscore versions are deprecated.