Chromium Code Reviews
Help | Chromium Project | Sign in
(661)

Issue 28823003: WIP: Disable warning C4996 for Win SDK 8.1 for all (Closed)

Can't Edit
Can't Publish+Mail
Start Review
Created:
1 year, 10 months ago by Yosi_UTC9
Modified:
1 year, 5 months ago
Reviewers:
cpu, scottmg, adibu2008, Will Harris
CC:
chromium-reviews
Visibility:
Public.

Description

Disable warning C4996 for Win SDK 8.1 This patch disables warning C4996 for MSVS2013, which is caused by using deprecated Windows API GetVersion() and GetVersionEx(). BUG=311488 TEST=n/a; no behavior changes

Patch Set 1 #

Total comments: 2

Messages

Total messages: 16 (0 generated)
Will Harris
https://codereview.chromium.org/28823003/diff/1/build/common.gypi File build/common.gypi (right): https://codereview.chromium.org/28823003/diff/1/build/common.gypi#newcode4394 build/common.gypi:4394: 'msvs_disabled_warnings': [4996] this turns off warnings for all deprecated ...
1 year, 10 months ago (2013-10-18 11:37:57 UTC) #1
Yosi_UTC9
On 2013/10/18 11:37:57, Will Harris wrote: > https://codereview.chromium.org/28823003/diff/1/build/common.gypi > File build/common.gypi (right): > > https://codereview.chromium.org/28823003/diff/1/build/common.gypi#newcode4394 ...
1 year, 10 months ago (2013-10-18 12:44:05 UTC) #2
scottmg
I didn't encounter these warnings when building on VS2013, http://crbug.com/288948. Can you paste the warnings ...
1 year, 10 months ago (2013-10-18 16:09:00 UTC) #3
scottmg
https://codereview.chromium.org/28823003/diff/1/build/common.gypi File build/common.gypi (right): https://codereview.chromium.org/28823003/diff/1/build/common.gypi#newcode4392 build/common.gypi:4392: ['MSVS_VERSION=="2013"', { needs 'or MSVS_VERSION=="2013e"' also. If it's mostly ...
1 year, 10 months ago (2013-10-18 16:11:56 UTC) #4
Yosi_UTC9
On 2013/10/18 16:09:00, scottmg wrote: > I didn't encounter these warnings when building on VS2013, ...
1 year, 10 months ago (2013-10-21 01:35:24 UTC) #5
Yosi_UTC9
1 year, 10 months ago (2013-10-21 01:35:37 UTC) #6
cpu
We haven't decided if we want to support the win8.1 sdk, moving from win7 sdk ...
1 year, 10 months ago (2013-10-21 18:01:30 UTC) #7
scottmg
(Please update the CL description to say 8.1 SDK rather than VS2013 too)
1 year, 10 months ago (2013-10-21 18:03:27 UTC) #8
Yosi_UTC9
On 2013/10/21 18:01:30, cpu wrote: > We haven't decided if we want to support the ...
1 year, 10 months ago (2013-10-22 03:49:57 UTC) #9
Will Harris
On 2013/10/22 03:49:57, Yoshifumi Inoue wrote: > On 2013/10/21 18:01:30, cpu wrote: > > We ...
1 year, 10 months ago (2013-10-22 09:13:41 UTC) #10
cpu
Ok lets do this. lgtm This does not mean we are endorsing the use of ...
1 year, 10 months ago (2013-10-24 00:29:51 UTC) #11
Yosi_UTC9
Is there way to check Win SDK version in gyp file? MSVS=2013 doesn't imply Win ...
1 year, 10 months ago (2013-10-24 01:44:16 UTC) #12
scottmg
On 2013/10/24 01:44:16, Yoshifumi Inoue wrote: > Is there way to check Win SDK version ...
1 year, 10 months ago (2013-10-24 02:19:29 UTC) #13
Yosi_UTC9
scottmg@, thanks for information. I'll write has_str.py then ['OS=="win" and chromium_code==0 and "<!(python <(DEPTH)/build/has_str.py <_WIN32_WINNT_WINBLUE ...
1 year, 10 months ago (2013-10-24 04:21:04 UTC) #14
scottmg
On 2013/10/24 04:21:04, Yoshifumi Inoue wrote: > scottmg@, thanks for information. > > I'll write ...
1 year, 10 months ago (2013-10-24 04:26:33 UTC) #15
adibu2008_gmail.com
1 year, 5 months ago (2014-03-31 23:11:31 UTC) #16

On Sunday, October 20, 2013 9:35:37 PM UTC-4, yo...@chromium.org wrote:
>
> https://codereview.chromium.org/28823003/ 
>

To unsubscribe from this group and stop receiving emails from it, send an email
to chromium-reviews+unsubscribe@chromium.org.
Sign in to reply to this message.

Powered by Google App Engine
RSS Feeds Recent Issues | This issue
This is Rietveld 020f378