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

Issue 11564017: Complete the transition to unicode APIs on Windows. (Closed)

Created:
8 years ago by Mads Ager (google)
Modified:
8 years ago
CC:
reviews_dartlang.org
Visibility:
Public.

Description

Complete the transition to unicode APIs on Windows. Define _UNICODE and UNICODE so that we do not add accidental dependencies on non-unicode APIs. Reapply the change without the completely broken code in socket_win.cc. Instead use 'A' versions explicitly. R=sgjesse@google.com,antonm@google.com Committed: https://code.google.com/p/dart/source/detail?r=16122

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+56 lines, -51 lines) Patch
M runtime/bin/extensions_win.cc View 1 chunk +4 lines, -1 line 0 comments Download
M runtime/bin/process_win.cc View 7 chunks +40 lines, -39 lines 0 comments Download
M runtime/bin/socket_win.cc View 2 chunks +10 lines, -10 lines 0 comments Download
M runtime/platform/globals.h View 1 chunk +2 lines, -1 line 0 comments Download

Messages

Total messages: 3 (0 generated)
Mads Ager (google)
8 years ago (2012-12-13 15:50:38 UTC) #1
Søren Gjesse
lgtm
8 years ago (2012-12-13 15:51:30 UTC) #2
Anton Muhin
8 years ago (2012-12-13 15:51:38 UTC) #3
lgtm

Powered by Google App Engine
This is Rietveld 408576698