Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(643)

Issue 12210033: Simplify secure API functions workaround for MinGW (Closed)

Created:
7 years, 10 months ago by Net147
Modified:
7 years, 10 months ago
Reviewers:
ulan, Yang
CC:
v8-dev
Visibility:
Public.

Description

Simplify secure API functions workaround for MinGW If MINGW_HAS_SECURE_API is defined, the definitions of localtime_s, fopen_s, _vsnprintf_s, strncpy_s had to be renamed using #define so they would not conflict with definitions in the standard headers. A simpler way to handle this is to undefine MINGW_HAS_SECURE_API before including the standard headers. Contributed by net147@gmail.com BUGS= TEST= Committed: https://code.google.com/p/v8/source/detail?r=13607

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+11 lines, -12 lines) Patch
M src/platform-win32.cc View 3 chunks +11 lines, -12 lines 0 comments Download

Messages

Total messages: 3 (0 generated)
Net147
7 years, 10 months ago (2013-02-06 12:59:33 UTC) #1
Yang
On 2013/02/06 12:59:33, Net147 wrote: LGTM.
7 years, 10 months ago (2013-02-06 13:03:22 UTC) #2
Yang
7 years, 10 months ago (2013-02-06 13:04:07 UTC) #3
Message was sent while issue was closed.
Committed manually as r13607 (presubmit successful).

Powered by Google App Engine
This is Rietveld 408576698