Chromium Code Reviews
[email protected] (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(370)

Issue 9418029: Roll icu to 122609 (Closed)

Created:
8 years, 10 months ago by jungshik at Google
Modified:
8 years, 10 months ago
Reviewers:
Nico
CC:
chromium-reviews
Visibility:
Public.

Description

Roll icu (122379 -> 122609) This is to make clang happy. See http://codereview.chromium.org/9420032/ for the actual changes. See also http://codereview.chromium.org/9355015/ (to suppress an unknown pragam warning in VS) BUG=84851, 92756 TEST=Builds on all platforms go fine and clang does not complain about decNumber.c and colldata.cpp in ICU. TBR=thakis Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=122611

Patch Set 1 #

Patch Set 2 : #

Unified diffs Side-by-side diffs Delta from patch set Stats (+1 line, -1 line) Patch
M DEPS View 1 1 chunk +1 line, -1 line 0 comments Download

Messages

Total messages: 2 (0 generated)
jungshik at Google
8 years, 10 months ago (2012-02-17 01:25:19 UTC) #1
Nico
8 years, 10 months ago (2012-02-17 01:27:48 UTC) #2
The numbers in the description is the wrong way round. You'll also have to sync
since I rolled to 122379 earlier today :-)

Powered by Google App Engine
This is Rietveld 408576698